Skip to content
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

Digests AUTH48 #2693

Merged
merged 61 commits into from
Feb 14, 2024
Merged

Digests AUTH48 #2693

merged 61 commits into from
Feb 14, 2024

Conversation

LPardue
Copy link
Contributor

@LPardue LPardue commented Dec 2, 2023

This is just the initial tranche of trivial changes. There's more to come as we work through some of the more complicated stuff. Will remain in draft state until the RFC is published.

  • Remove code samples and changelog
  • Section title nits
  • nits in section 1
  • nits in section 2+3
  • nits in section 4
  • nits in section 5
  • nits in section 6
  • nits in section 7
  • nit in refs
  • nits in examples
  • nits in remaining appendicies
  • nit: one more 3230 ref

@LPardue
Copy link
Contributor Author

LPardue commented Dec 10, 2023

The diff is available at https://author-tools.ietf.org/api/iddiff?url_1=https://www.rfc-editor.org/authors/rfc9530.txt&url_2=https://httpwg.github.io/http-extensions/digest-auth48/draft-ietf-httpbis-digest-headers.txt

The diff should be as minimal as we can get it (some things are simply can't get the same, but they don't matter)

@LPardue LPardue marked this pull request as ready for review February 14, 2024 19:56
@LPardue LPardue merged commit 6131253 into main Feb 14, 2024
2 checks passed
@LPardue LPardue deleted the digest-auth48 branch February 14, 2024 19:56
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.

None yet

1 participant