You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for this context. If more options are likely to added, I'd prefer to see this as an enum like class instead of an actual enum so that it's more extensible if options or props need to be added to potential auth types.
#527
Closed
mrpackethead opened this issue
Aug 9, 2023
· 1 comment
.....I'd prefer to see this as an enum like class instead of an actual enum so that it's more extensible if options or props need to be added to potential auth types.
Have replaced using the AuthType ENUM in favor of a Authorizer Class, which today, only returns the type of authorizer ( iam or none ), but will provide a non breaking path for any future auth models that may appear.
The text was updated successfully, but these errors were encountered:
.....I'd prefer to see this as an enum like class instead of an actual enum so that it's more extensible if options or props need to be added to potential auth types.
Originally posted by @TheRealAmazonKendra in #506 (comment)
Have replaced using the AuthType ENUM in favor of a Authorizer Class, which today, only returns the type of authorizer ( iam or none ), but will provide a non breaking path for any future auth models that may appear.
The text was updated successfully, but these errors were encountered: