charts icon indicating copy to clipboard operation
charts copied to clipboard

[artifactory] Prevent erroneous redirects to port 8443

Open TomyLobo opened this issue 5 months ago • 2 comments

  • /artifactory/ is currently (mis)handled by nginx and redirects to port 8443
  • Same with /artifactory, however, if I remove only the rewrite line, the request goes somewhere else (I think it's the router) and still redirects to port 8443.

This commit fixes both issues by letting Artifactory itself handle those redirects.

PR Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • [ ] Chart Version bumped
  • [ ] CHANGELOG.md updated
  • [x] Variables and other changes are documented in the README.md
  • [x] Title of the PR starts with chart name (e.g. [artifactory])

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes # Case: 357670

Special notes for your reviewer:

TomyLobo avatar Jul 16 '25 17:07 TomyLobo

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request

github-actions[bot] avatar Jul 16 '25 17:07 github-actions[bot]

Thanks for the PR! We'll review this internally and have created an internal tracking issue for it.

chukka avatar Jul 20 '25 05:07 chukka