-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[TT-11426/TT-13322] Add deprecation notice for external OAuth middleware #6657
base: master
Are you sure you want to change the base?
[TT-11426/TT-13322] Add deprecation notice for external OAuth middleware #6657
Conversation
💔 The detected issue is not in one of the allowed statuses 💔
Please ensure your jira story is in one of the allowed statuses |
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
API Changes --- prev.txt 2024-10-22 10:45:11.183495601 +0000
+++ current.txt 2024-10-22 10:45:04.967480956 +0000
@@ -1299,6 +1299,10 @@
Enabled bool `bson:"enabled" json:"enabled"`
Providers []Provider `bson:"providers" json:"providers"`
}
+ ExternalOAuth support will be deprecated starting from
+ 5.7.0. To avoid any disruptions, we recommend that
+ you use JSON Web Token (JWT) instead, as explained in
+ https://tyk.io/docs/basic-config-and-security/security/authentication-authorization/ext-oauth-middleware/
type GWStats struct {
APIsCount int `json:"apis_count"`
@@ -3352,7 +3356,11 @@
// Providers is used to configure OAuth providers.
Providers []OAuthProvider `bson:"providers" json:"providers"` // required
}
- ExternalOAuth holds configuration for an external OAuth provider.
+ ExternalOAuth holds configuration for an external OAuth
+ provider. ExternalOAuth support will be deprecated starting
+ from 5.7.0. To avoid any disruptions, we recommend that
+ you use JSON Web Token (JWT) instead, as explained in
+ https://tyk.io/docs/basic-config-and-security/security/authentication-authorization/ext-oauth-middleware/
type ExtractCredentialsFromBody struct {
// Enabled activates extracting credentials from body. |
PR Code Suggestions ✨Explore these optional code suggestions:
|
Quality Gate failedFailed conditions |
@@ -1523,6 +1523,9 @@ var Template = template.New("").Funcs(map[string]interface{}{ | |||
}, | |||
}) | |||
|
|||
// ExternalOAuth support will be deprecated starting from 5.7.0. | |||
// To avoid any disruptions, we recommend that you use JSON Web Token (JWT) instead, | |||
// as explained in https://tyk.io/docs/basic-config-and-security/security/authentication-authorization/ext-oauth-middleware/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// as explained in https://tyk.io/docs/basic-config-and-security/security/authentication-authorization/ext-oauth-middleware/ | |
// as explained in https://tyk.io/docs/basic-config-and-security/security/authentication-authorization/ext-oauth-middleware/. |
godoc trailing doc
User description
Description
Related Issue
Parent: https://tyktech.atlassian.net/browse/TT-11426
Subtask: https://tyktech.atlassian.net/browse/TT-13322
Motivation and Context
How This Has Been Tested
Screenshots (if appropriate)
Types of changes
Checklist
PR Type
enhancement, documentation
Description
EnabledForSpec
function.Changes walkthrough 📝
mw_external_oauth.go
Add deprecation warning for external OAuth middleware
gateway/mw_external_oauth.go