-
Notifications
You must be signed in to change notification settings - Fork 299
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
uid Invalid value #269
Comments
cc/ @squio |
Hmm looks like this PR broke more than it solved, please feel free to undo this merge (I have found a different work around which definitely won't break either dj-rest-auth or allauth: see https://gist.github.com/squio/28aadec528bea9302c168744fd0af3b6 ) |
I'm seeing a similar issue. Getting errors like the below on version
Edit: The issue goes away when I downgrade to |
@RommelTJ thanks for sharing your tmp solution to downgrade to 2.1.5; you saved me a lot of time! |
Please see my pr: #276 |
On 2.1.9 UID is still invalid & 2.11 didn't work. Downgrading to 2.1.5 worked for me, hoping this gets fixed asap as this effectively makes password reset unusable & wasn't a terribly great experience trying to implement this the first time :( |
@Tobeyforce any luck? |
I believe I ended up staying on 2.11 but instead overriding a serializer or view. I'm not entirely sure, looking at my prod & local dev I'm at 2.11 at least. It could be that the problem was on the allauth side for me. Wish I could help you out more here & wish I had followed up once I got things working. It could also be that there were some things that needed to be adjusted in urls.py. Things got quite complicated for me as I ended up using social auth as well which means I went back and forth between allauth docs and dj-rest-auth docs to make all things click. |
The same problem happened to me and it gets solved when downgrading to 2.1.5. It is related to the allauth import I think. Is there any solution apart from downgrading or not yet? |
@mateokurti Which version of dj-rest-auth are you using? I haven't tried past version 2.1.7, but this MR got merged so perhaps it's worth a try on the latest release? |
@RommelTJ I was upgrading all dependencies of my project to latest, so I tried the latest version first and it didn't work. Then tried downgrading until 2.1.5 seemed to work. I actually have it sleightly different. Instead of:
I have:
Not sure if that has to do with the problem |
Also facing the same problem, why isn't this fixed?! |
Hi!
It seems like the last PR #266 broke the
/dj-rest-auth/password/reset/confirm/
endpoint. If I use the last version of the package with the uid and token on the url which arrives on my email after creating the password reset request, I getHowever, after commenting out the include from
allauth
indj_rest_auth/serializers.py
and usingurlsafe_base64_decode
instead, the password is successfully changedHere is the relevant line on my
urls.py
fileI'm fairly new to django and I just started using this package, so the error may be on my side.
The text was updated successfully, but these errors were encountered: