-
Notifications
You must be signed in to change notification settings - Fork 55
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
login failing "Email or password incorrect" #68
Comments
tagging @savantes1 for the most recent PR about login. |
I was having a login issue as well. However, it seems to be working on my end after I went into
Let me know if this helps. |
It worked for me. Thanks! |
Happy to accept the above fix as a PR if anyone would like to put one forth. |
hey @hfaran! I have a question about this project, where can i reach you, i have a question about how this works, can i get ur email or smth? |
@mrnhtyzgld Feel free to ask any questions here or open a new issue on Github if you have a different question |
I'll try to take a look at this today or tomorrow and, hopefully, put in a PR |
@hfaran you didnt return to my issue, can you return when available? Sorry for asking. |
So, clearly, when I said "today or tomorrow", I meant "next week" ;) I'm pretty sure the pull request I just submitted should resolve this problem |
#71 doesn't/shouldn't fix this issue, since changing f-strings to I'm not sure whether the underlying cause for this specific issue is since the author mentioned they tried urlencoding; maybe it wasn't done correctly? because afaik the only difference between manually string-encoding the form data and using a dict is whether the payload gets urlencoded behind the scenes by anyway, i submitted PR #72 to implement @EddieJ03's suggested fix (#68 (comment)). |
However, I have confirmed 5 ways from sunday that the email and password are correct. I experimented with adding urlencoding to the user and pass but that does not seem to be helping. Any ideas? Would really like to be able to use your library to script some of my piazza posts.
The text was updated successfully, but these errors were encountered: