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
Remove support for allowUnauthenticated
#8160
Remove support for allowUnauthenticated
#8160
Conversation
Codecov Report
@@ Coverage Diff @@
## v2 #8160 +/- ##
==========================================
+ Coverage 88.17% 88.20% +0.03%
==========================================
Files 248 248
Lines 9434 9405 -29
==========================================
- Hits 8318 8296 -22
+ Misses 1116 1109 -7
Continue to review full report at Codecov.
|
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.
@morgan-sam In general it looks great. Code-wise I have just one remark towards documentation update.
Still all v2 proposals should be based against v2
branch, can you rebase your branch against it (I've already updated PR to show changes against this branch) ?
Thank you!
docs/deprecations.md
Outdated
@@ -32,7 +32,7 @@ Please upgrade to use at least Node.js v10 (It's recommended to use LTS version, | |||
|
|||
## AWS ALB `allowUnauthenticated` | |||
|
|||
Please use `onUnauthenticatedRequest` instead. `allowUnauthenticated` will be removed with v2.0.0 | |||
Please use `onUnauthenticatedRequest` instead. `allowUnauthenticated` has been deprecated |
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.
Let's not update deprecation notes.
allowUnauthenticated
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.
Thank you @morgan-sam looks great!
BREAKING CHANGE: Support for `providers.alb.authorizers[].allowUnauthenticated` setting was removed. Rely on `providers.alb.authorizers[].onUnauthenticatedRequest` instead. (PR #8160)
BREAKING CHANGE: Support for `providers.alb.authorizers[].allowUnauthenticated` setting was removed. Rely on `providers.alb.authorizers[].onUnauthenticatedRequest` instead. (PR #8160)
BREAKING CHANGE: Support for `providers.alb.authorizers[].allowUnauthenticated` setting was removed. Rely on `providers.alb.authorizers[].onUnauthenticatedRequest` instead. (PR #8160)
Closes: #8120
Removes allowUnauthenticated and allowUnauthenticated related tests.