8000 🐛 Bug Report: Phone authentication env variable not parsing slash · Issue #5913 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

🐛 Bug Report: Phone authentication env variable not parsing slash #5913

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
2 tasks done
Mr-Malomz opened this issue Jul 30, 2023 · 2 comments
Closed
2 tasks done

🐛 Bug Report: Phone authentication env variable not parsing slash #5913

Mr-Malomz opened this issue Jul 30, 2023 · 2 comments
Assignees
@stnguyen90
Labels
bug Something isn't working product / auth Fixes and upgrades for the Appwrite Auth / Users / Teams services.
Milestone
1.4.0

Comments

@Mr-Malomz
Copy link
Mr-Malomz commented Jul 30, 2023 •
edited by stnguyen90
Loading

👟 Reproduction steps

When I configure _APP_SMS_PROVIDER e.g. Telesign. Use the Create CreatePhoneSession() method in a web application.

👍 Expected behavior

Create a session for the inputted phone number and receive an OTP to complete registration.

👎 Actual Behavior

When the Create CreatePhoneSession() method is used, it returns an error.

Because the API Key used has a slash as part of the value, the system returns an InvalidArgumentException: Unable to parse DSN error.

Full error below

sms://E3CE8D57-97AD-XXXXXXXXXX:CNaceK4pHreLA1+et9nBIL8QhRUWdoTa6pi/qzk0D4HWXEjBaMcr15DHeUzXXXXXXXXXXX@telesign in /usr/src/code/src/Appwrite/DSN/DSN.php:54
Stack trace:

#0 /usr/src/code/app/workers/messaging.php(33): Appwrite\DSN\DSN->__construct('sms://E3CE8D57-...')

#1 /usr/src/code/src/Appwrite/Resque/Worker.php(98): MessagingV1->init()

#2 /usr/src/code/vendor/resque/php-resque/lib/Resque/Job.php(198): Appwrite\Resque\Worker->setUp()

#3 /usr/src/code/vendor/resque/php-resque/lib/Resque/Worker.php(303): Resque_Job->perform()

#4 /usr/src/code/vendor/resque/php-resque/lib/Resque/Worker.php(242): Resque_Worker->perform(Object(Resque_Job))

#5 /usr/src/code/vendor/resque/php-resque/bin/resque(185): Resque_Worker->work('1', false)

#6 {main}

🎲 Appwrite version

Version 1.3.x

💻 Operating system

MacOS

🧱 Your Environment

No response

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?
@Mr-Malomz Mr-Malomz added the bug Something isn't working label Jul 30, 2023
@stnguyen90 stnguyen90 self-assigned this Jul 30, 2023
@stnguyen90
Copy link
Contributor

@Mr-Malomz thanks for opening this! 🙏 Let me bring this up to the team to see how we can fix this.

@stnguyen90 stnguyen90 added this to the 1.4.0 milestone Jul 31, 2023
@stnguyen90 stnguyen90 mentioned this issue Jul 31, 2023
Merged
2 tasks
@github-actions github-actions bot mentioned this issue Aug 1, 2023
Closed
@joeyouss
Copy link
joeyouss commented Aug 7, 2023

Hi @Mr-Malomz the PR made to fix this is linked, we will close this issue soon, thank you for opening this! Appreciate the feedback!

@eldadfux eldadfux closed this as completed Sep 9, 2023
@eldadfux eldadfux added the product / auth Fixes and upgrades for the Appwrite Auth / Users / Teams services. label Sep 9, 2023
@github-actions github-actions bot mentioned this issue Oct 1, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stnguyen90 stnguyen90

Labels
bug Something isn't working product / auth Fixes and upgrades for the Appwrite Auth / Users / Teams services.
Projects
None yet
Milestone
1.4.0
Development

No branches or pull requests
4 participants
@eldadfux @stnguyen90 @Mr-Malomz @joeyouss
0