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.

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

change: violation uri-blocker.lua plugin response body remove attribute: rejected_msg #11533

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PGDream
Copy link

@PGDream PGDream commented Aug 27, 2024

the user define anthing content,whenever response body output to anything, don't add field "error_msg"

Description

The Response body output of the user define content。

the user define anthing content,whenever response body output to anything, don't add field "error_msg"
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. plugin labels Aug 27, 2024
Copy link
Contributor

@shreemaan-abhishek shreemaan-abhishek left a comment

Choose a reason for hiding this comment

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

please update the PR title, and I suppose there will be failing test cases. Please look at them too.

@PGDream PGDream changed the title Update uri-blocker.lua Violation uri-blocker.lua plugin response body remove attribute: rejected_msg Sep 10, 2024
@PGDream
Copy link
Author

PGDream commented Sep 10, 2024

done,please you look at

@PGDream PGDream changed the title Violation uri-blocker.lua plugin response body remove attribute: rejected_msg change: violation uri-blocker.lua plugin response body remove attribute: rejected_msg Sep 10, 2024
@shreemaan-abhishek
Copy link
Contributor

there are more failing test cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants