Skip to content

XRAY-156690 - Rubygems native support - #860

Open
Phavya-jfrog wants to merge 5 commits into
jfrog:devfrom
Phavya-jfrog:feature/XRAY-156690-ruby-native
Open

XRAY-156690 - Rubygems native support#860
Phavya-jfrog wants to merge 5 commits into
jfrog:devfrom
Phavya-jfrog:feature/XRAY-156690-ruby-native

Conversation

@Phavya-jfrog

@Phavya-jfrog Phavya-jfrog commented Aug 26, 2026

Copy link
Copy Markdown
Contributor
  • The pull request is targeting the dev branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....
  • All static analysis checks passed.
  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • Updated the Contributing page / ReadMe page / CI Workflow files if needed.
  • All changes are detailed at the description. if not already covered at JFrog Documentation, new documentation have been added.

Add native repository resolution for RubyGems in jf ca.

RubyGems resolution priority (no jf ruby-config/yaml — always native):

  1. ~/.gemrc's Artifactory-shaped entry in :sources: — the only source ("Set me up")
  2. Clear error if it doesn't resolve

Screenshots attached in https://jfrog-int.atlassian.net/browse/XRAY-156690?focusedCommentId=1213539

gosec flags the literal user:password@host URL shape as a potential
hardcoded credential regardless of the value, so annotate the test
fixture (a fake placeholder, not a real secret) with #nosec.
z0jpd630304795.jfrogdev.org and rubygems-repo-jfca looked like a real
D2C environment and repo name left over from manual testing. Replace
with generic myrt.jfrogdev.org / rubygems-repo-test, matching the
placeholders already used by other tech tests (yarn, python, uv) and
by the gem repo tests in curationaudit_test.go.
@Phavya-jfrog Phavya-jfrog added the improvement Automatically generated release notes label Aug 26, 2026
@Phavya-jfrog
Phavya-jfrog force-pushed the feature/XRAY-156690-ruby-native branch from eb9e348 to 6a82cec Compare August 26, 2026 05:45
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Aug 26, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Aug 26, 2026
@Phavya-jfrog
Phavya-jfrog force-pushed the feature/XRAY-156690-ruby-native branch from 81089e6 to 6a82cec Compare August 26, 2026 06:59
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Aug 26, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Aug 26, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Aug 27, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Aug 27, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Aug 27, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Aug 27, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Aug 27, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Aug 27, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Aug 28, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Aug 28, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Aug 28, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Automatically generated release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant