Skip to content

Add algorithm parameter (Django 3.1) #489

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

Merged
merged 3 commits into from
Jan 20, 2021
Merged

Conversation

okapies
Copy link
Contributor

@okapies okapies commented Oct 14, 2020

Add algorithm parameter or field, which is introduced in Django 3.1, to the following functions/classes:

  • django.core.signing.Signer.__init__
  • django.core.signing.base64_hmac
  • django.utils.crypto.salted_hmac
  • django.contrib.auth.tokens.PasswordResetTokenGenerator

Related issues

@sobolevn
Copy link
Member

Thanks a lot! I am going to leave this hanging for a while 😞

Please, see #262

@okapies
Copy link
Contributor Author

okapies commented Oct 14, 2020

@sobolevn I see. I also think there might be a problem around the versioning. Please leave it until the issue is resolved.

@mkurnikov
Copy link
Member

mkurnikov commented Jan 19, 2021

@sobolevn I think it should be backwards compatible? The parameter is optional, the worst case will be a false-positive, which is fine. Right? Please merge it, if you agree with this.

@mkurnikov mkurnikov changed the title Add algorithm parameter Add algorithm parameter (Django 3.1) Jan 19, 2021
@sobolevn sobolevn merged commit bdc7db1 into typeddjango:master Jan 20, 2021
snejus pushed a commit to snejus/django-stubs that referenced this pull request Mar 10, 2021
* add algorithm parameter

* add algorithm field

* reformat
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants