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

Use external_url if exists on Add Repo Command chart details page #9189

Merged
merged 1 commit into from Oct 15, 2019

Conversation

strainovic
Copy link
Contributor

If we have https reverse proxy in a front of harbor, we can set external_url and harbor will use it but not on chart details page within Add Repo command (it will be http://) if you have https -> http proxy setup.

Signed-off-by: strainovic <nenad.strainovic@gmail.com>
@coveralls
Copy link

coveralls commented Sep 20, 2019

Pull Request Test Coverage Report for Build 15595

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 66.163%

Totals Coverage Status
Change from base Build 15592: 0.0%
Covered Lines: 22617
Relevant Lines: 34184

💛 - Coveralls

Copy link
Member

@ninjadq ninjadq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@AllForNothing AllForNothing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AllForNothing AllForNothing merged commit 285d3e1 into goharbor:master Oct 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants