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 True for all the post and get inputs #2015

Closed
wants to merge 1 commit into from

Conversation

enferas
Copy link

@enferas enferas commented Dec 29, 2022

Signed-off-by: AL-KASSAR f.al-kassar@sap.com

I added TRUE for all the inputs, then all the inputs will be sanitized against XSS vulnerabilities. POST in CI

Signed-off-by: AL-KASSAR <f.al-kassar@sap.com>
@Pr3d4dor
Copy link
Collaborator

Thanks @enferas, I would suggest to change all "TRUE" to "true" to maintain the PSR2 and PSR12 standards.

@github-actions
Copy link

Esta PR está obsoleta porque está aberta há 30 dias sem atividade. Remova o rótulo ou esse comentário ou essa issue será fechada em 5 dias

@github-actions github-actions bot added the Stale label Feb 26, 2023
@github-actions github-actions bot closed this Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants