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
fix: cookies filter secure invalid #37203
Conversation
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.
Can you please describe the issue in more detail as well as properly fill out the PR template? Additionally, we should have some tests for this.
711a629
to
c954841
Compare
@codebytere Description updated and already add tests. |
@BlackHole1 can you update the |
Co-authored-by: Robo <hop2deep@gmail.com>
@deepak1556 Done. I updated the |
Release Notes Persisted
|
I have automatically backported this PR to "24-x-y", please check out #37246 |
I have automatically backported this PR to "22-x-y", please check out #37247 |
I have automatically backported this PR to "23-x-y", please check out #37248 |
Description of Change
gist: https://gist.github.com/6d2ba80a835a75f90f385a2f4664a6a0
This is a simple judgment condition error that just needs to be corrected to the correct judgment condition.
Checklist
npm test
passesRelease Notes
Notes: Fixed an incorrect result returned when using secure as a cookies filter.