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

[Code Analysis] Report by sonarcloud.io #788

Open
wing328 opened this issue Aug 11, 2018 · 4 comments
Open

[Code Analysis] Report by sonarcloud.io #788

wing328 opened this issue Aug 11, 2018 · 4 comments

Comments

@wing328
Copy link
Member

wing328 commented Aug 11, 2018

Description

The code analysis report by https://sonarcloud.io/project/issues?id=org.openapitools%3Aopenapi-generator-project&resolved=false&types=BUG uncovers some bugs and vulnerabilities.

openapi-generator version

Latest master

Suggest a fix/enhancement

@ymohdriz is very kind to offer help to fix the bugs in the report. If anyone wants to help out (e.g. fixing the vulnerabilities), please reply to let us know.

@jmini
Copy link
Member

jmini commented Aug 15, 2018

Does this mean that someone registered us at sonarcloud? We have issue #33 pending since month...

I think that we can also integrate "sonarcloud.io" with GitHub PRs to get a report of the changes introduced by a PR. We should also investigate this.

@wing328
Copy link
Member Author

wing328 commented Aug 15, 2018

Does this mean that someone registered us at sonarcloud? We have issue #33 pending since month...

I submitted openapi-generator to sonarcloud to get the report (free for open source repo) and yes, it's related to the discussion in #33

I think that we can also integrate "sonarcloud.io" with GitHub PRs to get a report of the changes introduced by a PR. We should also investigate this.

Definitely (to avoid running the report/check manually)

@artwo
Copy link
Contributor

artwo commented Oct 19, 2021

Hi all,
I would like to collaborate in this open issue. There are a few improvements to make related to the Sonarcloud reported issues. I'm going to open a PR for this.

@artwo
Copy link
Contributor

artwo commented Oct 19, 2021

Added new PR #10641. I also checked out all of the issues in the sonarcloud report, so this issue can be close after merging the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants