Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix RequestParameterPolicyEnforcementFilter configuration logic for 7.0.x branch #5992

Merged
merged 1 commit into from
Feb 9, 2024
Merged

Fix RequestParameterPolicyEnforcementFilter configuration logic for 7.0.x branch #5992

merged 1 commit into from
Feb 9, 2024

Conversation

rrenomeron
Copy link
Contributor

master: #5938

  • Fix "parameters-to-check" property parsing to be comma-delimited so the default property value will work as expected
  • Allow characters below 0x20 to be blocked by the filter by parsing the "characters-to-forbid" property as delimited only by the space character, rather than by any whitespace character
  • Update unit tests as needed.

@rrenomeron
Port apereo#5938 to 7.0.x branch
0b801c6 
- Fix "parameters-to-check" property parsing to be comma-delimited so the default property value will work as expected
- Allow characters below 0x20 to be blocked by the filter by parsing the "characters-to-forbid" property as delimited only by the space character, rather than by any whitespace character
- Update unit tests as needed.
@apereocas-bot apereocas-bot added this to the 7.0.2 milestone Feb 9, 2024
@apereocas-bot apereocas-bot added Configuration Auto Merge Merge pull requests submitted via automated bots, such as RenovateBot, etc labels Feb 9, 2024
apereocas-bot
apereocas-bot approved these changes Feb 9, 2024
View reviewed changes
Copy link
Contributor

@apereocas-bot apereocas-bot left a comment

Choose a reason for hiding this comment

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

This pull request is approved and it will be merged shortly. Thank you very much! 🎉🎉

image

@apereocas-bot apereocas-bot merged commit 80bc7be into apereo:7.0.x Feb 9, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apereocas-bot apereocas-bot apereocas-bot approved these changes

Assignees
No one assigned
Labels
Auto Merge Merge pull requests submitted via automated bots, such as RenovateBot, etc Configuration Remember: Port Forward
Projects
None yet
Milestone
7.0.2
2 participants
@rrenomeron @apereocas-bot