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

Project Name Bug regarding the CNES Report #319

Closed
muhyid opened this issue Jun 6, 2022 · 2 comments · Fixed by #324
Closed

Project Name Bug regarding the CNES Report #319

muhyid opened this issue Jun 6, 2022 · 2 comments · Fixed by #324
Labels
bug Something isn't working done This issue was solved but not merged in master
Milestone

Comments

@muhyid
Copy link

muhyid commented Jun 6, 2022

Describe the bug

It seems if the project name contains ":" there is a problem.

My Environment:
Windows Server 2016
SonarQube 8.9.8 LTS Community Edition
Java 11.0.13 LTS

We are using the ProjectKey:1.0.0 format for Project Names. When I try to get a report from the project that is named like this. CNES report is running successfully and It says "Report generation: SUCCESS", But it is not producing any report file and occurring only 1 file with 0 bytes.
If I change the Project name like the ProjectKey-1.0.0 format. It is running properly and producing all report files
There is a log for problematic ProjectKey:1.0.0 format in the following. The result is exactly the same for the successful ones.
SonarQube URL: http://localhost:9000
SonarQube online: true
Detected SonarQube version: 8.9.8.54436
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.apache.poi.openxml4j.util.ZipSecureFile$1 (file:/C:/sonarqube/extensions/plugins/sonar-cnes-report-4.1.1.jar) to field java.io.FilterInputStream.in
WARNING: Please consider reporting this to the maintainers of org.apache.poi.openxml4j.util.ZipSecureFile$1
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
Report generation: SUCCESS

To reproduce

Steps to reproduce the behavior.

  1. Please create a project with ABC-DEF:1.0.0 format
  2. Please run CNES report
  3. Check the zip file
  4. Please create a project with ABC-DEF-1.0.0 format
  5. Please run CNES report
  6. Check the zip file
@muhyid muhyid added the bug Something isn't working label Jun 6, 2022
@Sancretor
Copy link
Contributor

Hi @muhyid
Thanks for your feedback, and sorry for this issue.
I'll have a look into it and will hopefully give you some feedback quickly.

@Sancretor Sancretor added investigation This issue is being investigated and removed investigation This issue is being investigated labels Jun 8, 2022
@Sancretor
Copy link
Contributor

Obviously, Windows does not accept ":" in filenames, and as we mostly work on Linux machines, we did not see this issue.
I'll make a fix for this, and it will be packaged inside the next version...
Unfortunately, I can't tell you exactly when it'll be released though.

@Sancretor Sancretor added this to the 4.2.0 milestone Jun 8, 2022
@Sancretor Sancretor mentioned this issue Jun 8, 2022
12 tasks
@Sancretor Sancretor added the done This issue was solved but not merged in master label Jun 8, 2022
@Sancretor Sancretor mentioned this issue Jun 13, 2022
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working done This issue was solved but not merged in master
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants