Skip to content

Fix release repo path construction - #940

Merged
eranturgeman merged 4 commits into
jfrog:devfrom
eranturgeman:fix-release-repo-path-construction
Nov 8, 2026
Merged

eranturgeman merged 4 commits into
jfrog:devfrom
eranturgeman:fix-release-repo-path-construction

Conversation

@eranturgeman

@eranturgeman eranturgeman commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Following a report from a customer, the remote repo URL construction was incorrect for artifacts sitting directly under the root (and not inside <bin_repo>/artifactory/...)
This PR resolves the issue, supporting both cases without breaking any existing customer by implementing a mechanism that attempts both URL with a retry mechanism if the first failed

@coderabbitai

coderabbitai Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

Explain Change →

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI; Enforced overrides: Organization, Workspace
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: 50bb725f-7e57-419e-8f20-2c17366e76e1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

@eranturgeman
eranturgeman changed the base branch from main to dev November 8, 2026 10:36
@eranturgeman
eranturgeman requested a review from a team November 8, 2026 10:37
@eranturgeman eranturgeman added the safe to test Approve running integration tests on a pull request label Nov 8, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Nov 8, 2026
@eranturgeman eranturgeman added the bug Something isn't working label Nov 8, 2026
Comment thread utils/utils.go Resolved
Comment thread utils/utils.go Resolved
Comment thread utils/paths.go Resolved
@eranturgeman eranturgeman added the safe to test Approve running integration tests on a pull request label Nov 8, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Nov 8, 2026
@github-actions

github-actions Bot commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

👍 Frogbot scanned this pull request and did not find any new security issues.


@eranturgeman
eranturgeman merged commit d19dab5 into jfrog:dev Nov 8, 2026
67 checks passed

This branch was successfully deployed

1 active deployment
frogbot — 5d782e3d Deployed Nov 8, 2026 by eranturgeman via scan-pull-request #4782
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 this pull request may close these issues.

2 participants