Skip to content

Commit

Permalink
Deprecate AuthConfig pluginAuth in extauth.proto (#10258)
Browse files Browse the repository at this point in the history
Co-authored-by: soloio-bulldozer[bot] <48420018+soloio-bulldozer[bot]@users.noreply.github.com>
Co-authored-by: changelog-bot <changelog-bot>
Co-authored-by: Rachael Graham <[email protected]>
  • Loading branch information
3 people authored Nov 6, 2024
1 parent 202a645 commit 90f91ad
Show file tree
Hide file tree
Showing 6 changed files with 1,362 additions and 1,349 deletions.
5 changes: 5 additions & 0 deletions changelog/v1.18.0-beta33/deprecate-AuthConfig-pluginAuth.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
changelog:
- type: NON_USER_FACING
description: Mark AuthConfig spec.configs.pluginAuth as deprecated in favor of passThroughAuth.
issueLink: https://github.com/solo-io/solo-projects/issues/6961
resolvesIssue: false

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -700,7 +700,7 @@ message ExtAuthConfig {
BasicAuth basic_auth = 4;
BasicAuthInternal basic_auth_internal = 17;
ApiKeyAuthConfig api_key_auth = 5;
AuthPlugin plugin_auth = 6;
AuthPlugin plugin_auth = 6 [deprecated = true];
OpaAuthConfig opa_auth = 7;
Ldap ldap = 8 [deprecated = true];
// Used for LDAP configurations that need service account credentials saved in a secret.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ message AuthConfig {
OAuth oauth = 2 [deprecated = true];
OAuth2 oauth2 = 8;
ApiKeyAuth api_key_auth = 4;
AuthPlugin plugin_auth = 5;
AuthPlugin plugin_auth = 5 [deprecated = true];
OpaAuth opa_auth = 6;
Ldap ldap = 7;
// This is a "dummy" extauth service which can be used to support multiple auth mechanisms with JWT authentication.
Expand Down Expand Up @@ -247,6 +247,7 @@ message CustomAuth {
string name = 2;
}

// **Deprecated**: The pluginAuth config type is deprecated and will be removed in a future release. Use passThroughAuth instead.
message AuthPlugin {
// Name of the plugin
string name = 1;
Expand Down
Loading

0 comments on commit 90f91ad

Please sign in to comment.