Veracode Batch Fix#845
Open
github-actions[bot] wants to merge 17 commits into
Open
Conversation
…ler/UserController.java with patch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
VERACOE-FIX CODE SUGGESTIONS
Caution
Breaking Flaws identified in code!
Fixes for src/main/java/com/veracode/verademo/controller/UserController.java:
Falws found for this file:
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 231 for issue 1136
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 495 for issue 1141
CWE 89 - Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection') - Severity 4 on line 253 for issue 1016
CWE 89 - Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection') - Severity 4 on line 508 for issue 1025
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 710 for issue 1135
CWE 89 - Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection') - Severity 4 on line 497 for issue 1017
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 114 for issue 1139
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 239 for issue 1155
CWE 89 - Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection') - Severity 4 on line 386 for issue 1010
CWE 89 - Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection') - Severity 4 on line 318 for issue 1015
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 660 for issue 1137
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 805 for issue 1143
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 257 for issue 1140
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 696 for issue 1148
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 387 for issue 1138
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 251 for issue 1149
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 506 for issue 1150
CWE 117 - Improper Output Neutralization for Logs - Severity 3 on line 861 for issue 1142
This PR is created by the Veracode-Fix bot to help fix security defects on your code
The base branch is fixtest the base commit sha is 12e045b
Please reach out to your Veracode team if anything in question