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

SonarQube Result-Writer fails: SonarQube-WebApi allows to request for a maximum of 10000 issues only #33

Open
l34d51n63r opened this issue Mar 10, 2017 · 5 comments

Comments

@l34d51n63r
Copy link

l34d51n63r commented Mar 10, 2017

The WriteFiles-Class fails, as SonarQube only allows to request for a maximum of 10000 issues. This could be fixed by building the request with the rules-parameter (&rules=squid:XXX), which will give us only the relevant issues. (SonarQube version 6.2)

@davewichers
Copy link
Contributor

Thanks for reporting. Another Benchmark user reported the same issue this week, along with some other details as we got farther along that we need to investigate, fix, and push out for SonarQube.

@minshi123
Copy link

hi, any progress here:-? (same questions.)

@davewichers
Copy link
Contributor

FYI. The SonarQube support in Benchmark is so old as to be fundamentally broken I suspect. Need to rewrite how this works. Ideally, SonarQube can simply export an XML results file now, and we can write a parser for it, like all the other tools. If anyone knows that SonarQube can export a results file, please explain how, and send me a Benchmark results file for SonarQube and I'm happy to write a parser for it. (Or you could do it yourself and submit a pull request :-) ).

@magussiro
Copy link

magussiro commented Aug 1, 2019

HI
I am trying to parsing SonarQubeJava rules sets to cwe, but i found the cwelookup method only transfer squid to cwe one on one, if there have easyway to parse SonarQubeJava rules sets?

here is the order sets
https://pastebin.com/r2SDQRzj

ref:

src/main/java/org/owasp/benchmark/score/parsers/SonarQubeReader.java

https://github.com/SonarSource/sonar-java/tree/35e70591626e1b27bb059f795dda0327c02a09d3/java-checks/src/main/resources/org/sonar/l10n/java/rules/squid

@davewichers
Copy link
Contributor

@darkspirit510 - Have you fixed this issue too?

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

No branches or pull requests

4 participants