Skip to content

Conversation

@jennifer-richards
Copy link
Contributor

Fixes#1313

Description of the Change

This updates RFC references that pointed at now-deprecated tools.ietf.org to use rfc-editor.org instead. Several links were updated from http to https. A reference in oauth2_validators.py was still referring to an old revision of the pre-RFC draft, so I've updated that to the corresponding section of the published RFC (the text in that section looked unchanged at a glance).

Checklist

  • PR only contains one change (considered splitting up PR)
  • unit-test added
  • documentation updated
  • CHANGELOG.md updated (only for user relevant changes)
  • author name in AUTHORS

@jennifer-richards
Copy link
ContributorAuthor

n.b., I did not update URLs in the changelog on the theory that it's archival

Copy link
Contributor

@n2ygkn2ygk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@codecov
Copy link

codecovbot commented Sep 12, 2023

Codecov Report

Merging #1315 (80bbb04) into master (adcb276) will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #1315 +/- ## ======================================= Coverage 97.37% 97.37% ======================================= Files 32 32 Lines 2022 2022 ======================================= Hits 1969 1969 Misses 53 53 
Files ChangedCoverage Δ
oauth2_provider/generators.py100.00% <ø> (ø)
oauth2_provider/oauth2_validators.py93.98% <ø> (ø)
oauth2_provider/views/introspect.py100.00% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@n2ygkn2ygk merged commit f8c9f36 into django-oauth:masterSep 12, 2023
Sign up for freeto 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.

Outdated RFC link in README.rst and elsewhere

2 participants

@jennifer-richards@n2ygk