Skip to content

fix(Oauth2): incorrect RFC reference in OAuth2 plugin schema #14371

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dingjiayi
Copy link
Contributor

@dingjiayi dingjiayi commented Mar 19, 2025

Summary

This PR fixes incorrect references to the OAuth 2.0 specification in the OAuth2 plugin schema. The current implementation incorrectly references RFC 6742, while the OAuth 2.0 Authorization Framework is actually defined in RFC 6749.

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

@team-eng-enablement team-eng-enablement added the author/community PRs from the open-source community (not Kong Inc) label Mar 19, 2025
@bungle bungle force-pushed the fix/oauth2-rfc-reference branch from ee2a13d to f9ec365 Compare March 27, 2025 10:56
@dingjiayi
Copy link
Contributor Author

@bungle Please help add the label skip-changelog.

@dingjiayi dingjiayi force-pushed the fix/oauth2-rfc-reference branch from f9ec365 to 16ac4c8 Compare April 2, 2025 10:45
@dingjiayi
Copy link
Contributor Author

Hi @ProBrian Could you kindly review this when you have time?

@dingjiayi
Copy link
Contributor Author

Hi @bungle 👋, a friendly ping to let me know if there's anything I can clarify or adjust to move this forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author/community PRs from the open-source community (not Kong Inc) size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants