Skip to content

Conversation

@Kludex
Copy link
Member

@Kludex Kludex commented Apr 11, 2021

Closes #1476

Changes ✨ :

  • Add realm as specified 1.
  • Add optional algorithm header. If not present, should use MD5, as specified 1.

TODO:

  • Implement the changes based on RFC 7616.
  • Documentation.

Decisions

  • I didn't add the domain parameter on the HTTPDigest class, as it's optional and I've noticed that some implementations ignores its existence.

References

Footnotes:

1: https://tools.ietf.org/html/rfc7616#section-3.3

Updates

  • 2021-04-11: Add support to algorithm and realm.

@Kludex Kludex marked this pull request as draft April 11, 2021 19:23
@github-actions
Copy link
Contributor

πŸ“ Docs preview for commit 03ae0bb at: https://60734d3ba49b01106e569dea--fastapi.netlify.app

@github-actions
Copy link
Contributor

πŸ“ Docs preview for commit 6ef91a1 at: https://6073665bca631c51c79b0f87--fastapi.netlify.app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve HTTPDigest implementation

1 participant