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

Add flag support for params #142

Closed
spaze opened this issue Nov 25, 2022 · 0 comments · Fixed by #150
Closed

Add flag support for params #142

spaze opened this issue Nov 25, 2022 · 0 comments · Fixed by #150
Labels
enhancement New feature or request

Comments

@spaze
Copy link
Owner

spaze commented Nov 25, 2022

I'd like to allow params when a flag is present ia param value like JSON_HEX_TAG | JSON_HEX_APOS. Currently that's not possible and a param value is treated as scalar value.

This will need a new param format, one idea is mentioned in #141 (comment)

After adding the support, json_decode() with JSON_THROW_ON_ERROR could be added to the "loose" config as originally proposed by @ilazaridis in #141.

Also this should be updated too then

function: 'htmlspecialchars()'
message: 'set the $flags parameter to `ENT_QUOTES` to also convert single quotes to entities to prevent some HTML injection bugs'
allowParamsAnywhere:
2: ::ENT_QUOTES

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant