Skip to content

Conversation

@matiseni51
Copy link
Contributor

@matiseni51matiseni51 commented Sep 30, 2022

Fixes#1207

Description of the Change

code_challenge_method=S256 in authorize call added in getting_started documentation

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

@n2ygk
Copy link
Contributor

n2ygk commented Oct 4, 2022

@matiseni51 please fill out the pull request template (not sure how that got missed) and add it to the description.

@codecov
Copy link

codecovbot commented Oct 4, 2022

Codecov Report

Merging #1208 (d95d218) into master (9383e08) will decrease coverage by 0.81%.
The diff coverage is n/a.

@@ Coverage Diff @@## master #1208 +/- ## ========================================== - Coverage 96.85% 96.04% -0.82%  ========================================== Files 31 26 -5 Lines 1813 1314 -499 ========================================== - Hits 1756 1262 -494 + Misses 57 52 -5 
Impacted FilesCoverage Δ
oauth2_provider/admin.py
oauth2_provider/models.py
oauth2_provider/settings.py
oauth2_provider/urls.py
oauth2_provider/__init__.py

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

@matiseni51
Copy link
ContributorAuthor

@n2ygk sorry, my bad.

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!

@n2ygkn2ygk merged commit da459a1 into django-oauth:masterOct 4, 2022
@n2ygkn2ygk added this to the 2.2.0 milestone Oct 18, 2022
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.

Docs - code_verifier in authorization_code

3 participants

@matiseni51@n2ygk@tykling