Skip to content

fixes #349 UnifiedSecurityHandler should check each handler isEnabled…#350

Merged
stevehu merged 1 commit intomasterfrom
issue349
Feb 8, 2024
Merged

fixes #349 UnifiedSecurityHandler should check each handler isEnabled…#350
stevehu merged 1 commit intomasterfrom
issue349

Conversation

@stevehu
Copy link
Copy Markdown
Contributor

@stevehu stevehu commented Feb 7, 2024

… before calling it

@stevehu stevehu requested a review from DiogoFKT February 7, 2024 16:51
DiogoFKT
DiogoFKT reviewed Feb 7, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@DiogoFKT DiogoFKT left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since now we are always checking for JWT, SWT and APIKey isEnabled() I believe we should define the default of those properties as TRUE

@stevehu stevehu merged commit 500b708 into master Feb 8, 2024
@stevehu stevehu deleted the issue349 branch February 8, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DiogoFKT DiogoFKT DiogoFKT left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@stevehu @DiogoFKT
⚡