Skip to content

Conversation

@reshmifrog
Copy link
Contributor

  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • The pull request is targeting the master branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....

Problem : command jf rt curl is very fragile, regarding the order of curl args.
Solution: Refactor the findUriValueAndIndex() method to properly handle the argument value which was getting ignored previously.

@reshmifrog reshmifrog requested a review from bhanurp November 10, 2025 10:01
@reshmifrog reshmifrog added the bug Something isn't working label Nov 10, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Nov 10, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 10, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Nov 12, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 12, 2025
@reshmifrog reshmifrog changed the title updated core deps fix-curl-cmd-arg-parsing-issue Nov 12, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Nov 12, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 12, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Nov 12, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 12, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Nov 13, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 13, 2025
@github-actions
Copy link
Contributor

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


@reshmifrog reshmifrog merged commit a8ebc4f into jfrog:master Nov 14, 2025
72 of 81 checks passed
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.

3 participants