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.

Question

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.

mrpackethead opened this issue 9 months ago · comments

.....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.

Momo Kornher · Answer 1 · Sat Oct 14 2023 02:27:00 GMT+0800 (China Standard Time)

This seems to be created as a separate issue by mistake.