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

EC72 Java Example Code #361

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lbg-pete
Copy link

The EC72 example Java compliant code doesn't compile and breaks other SonarQube rules that would prevent introducing a vulnerability to SQL Injection.

This suggested contribution, if you agree, reworks the example complaint code so that it correctly compiles, conforms to other SonarQube rules and still results in the energy efficiency saving that the rule is intended to give.

mccorrip and others added 2 commits November 8, 2024 13:56
The EC72 example compliant code doesn't compile and breaks other SonarQube rules that prevent introducing vulnerability to SQL Injection.

This example complaint code conforms to other SonarQube rules, correctly compiles and still results in the efficiency saving that the rule is intended to give.
…CoE/EC72-Java-Fix

Update EC72 Example Code
@lbg-pete
Copy link
Author

Hi @dedece35, is there anything I've missed? This is my first OS contribution, so hoping I've followed the process correctly.

@utarwyn utarwyn requested a review from dedece35 December 11, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants