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

[GitHub][PR Decoration] Private Sonarqube server static images unavailable #130

Closed
Brialius opened this issue Mar 17, 2020 · 3 comments · Fixed by #150
Closed

[GitHub][PR Decoration] Private Sonarqube server static images unavailable #130

Brialius opened this issue Mar 17, 2020 · 3 comments · Fixed by #150
Labels
bug Something isn't working

Comments

@Brialius
Copy link
Contributor

Describe the bug
Github uses https://camo.githubusercontent.com/ to proxy any user images on their side and this cannot be disabled. It means every image should be available for GitHub proxy server, which is impossible with the way of serving static images #80 for private hosted Sonarqube server behind a strict firewall.

To Reproduce
Steps to reproduce the behavior:

  1. Setup CE Branch plugin on a private Sonarqube server
  2. Use GitHub as a provider
  3. Run Sonarqube branch check
  4. See error

Expected behavior
Static images are available in GitHub PR checks for private and public accessible Sonarqube servers

Screenshots
If applicable, add screenshots to help explain your problem.
image

image

Software Versions

  • SonarQube Version: Version 7.9.2 (build 30863)
  • Plugin Version: 1.3.0

Additional context
Please return back function of com.github.mc1arke.sonarqube.plugin.branch.image-url-base and make it optional or provide other way to solve this issue.

@Brialius Brialius added the bug Something isn't working label Mar 17, 2020
@don-vip
Copy link
Contributor

don-vip commented Mar 17, 2020

I guess a partial revert of #81 is needed to reintroduce the property I removed?

@choi-inkyun
Copy link

I have the same problem. The problem arises because the my sonarqube server is 'http' and github is 'https'.

@Brialius
Copy link
Contributor Author

I guess a partial revert of #81 is needed to reintroduce the property I removed?

Yeah, looks like that..

mc1arke pushed a commit that referenced this issue May 17, 2020
Some ALMs (such as Github) proxy images in Pull Request decoration and therefore need direct access to the images. If the Sonarqube server isn't internet facing, the ALM will not be able to retrieve the images, and they therefore wont be visible on the Pull Request. This change restores the property for allowing Sonarqube administrators to define the URL for retrieving images, with the images defaulting to loading from the Sonarqube if the property is not set/overridden.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants