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.
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
[bug] - all login params are required #11
Comments
Yes, actually we had to change the original implementation after browsers started blocking 3rd party cookies that we were using for sessions. I guess the code examples got out of sync, I will make sure to fix it. I will have to take a deeper look at the default param issue and see how we can fix it. |
Thanks. I may try to dig on this this evening or tomorrow. To clarify @eldadfux - I'm trying to call the appwrite SDK from JS, and am just trying to get back the 200 or 401 and the corresponding payload to process and move on from that login point. The payload coming back is always '400' because of these missing fields. |
We are trying to remove the extra params required for the login and register methods in this issue: #268, Hopefully this will work for modern browsers and we'll only use the extra params as fallback for older ones. |
…e/ide-helper-4.5.4 Bump swoole/ide-helper from 4.5.2 to 4.5.4
Calling from JS
aw.auth.login(this.username, this.password);
yields back
The 'success' and 'failure' URLs are still required by the backend.
This might be related to
utopia-php/http@5ac8c91#diff-58980ac23a3a9a0b92abd9805eed1e78
in Utopia forcing default values? It's a bit hard to follow as I'm not sure what 'param' method is doing here:
appwrite/app/controllers/auth.php
Line 35 in 6a5f546
Is param() call creating/invoking a validator directly?
The docs/examples are somewhat out of sync as well, as docs show
appwrite.users.login()
but it should be
appwrite.auth.login()
The text was updated successfully, but these errors were encountered: