Skip to content

Expose disable-token-refresh flag on jf login - #3643

Open
ehl-jf wants to merge 1 commit into
masterfrom
feature/JGC-520
Open

Expose disable-token-refresh flag on jf login#3643
ehl-jf wants to merge 1 commit into
masterfrom
feature/JGC-520

Conversation

@ehl-jf

@ehl-jf ehl-jf commented Aug 6, 2026

Copy link
Copy Markdown
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 ./....

Adds a --disable-token-refresh flag to jf login (JGC-520). Previously this setting was only reachable via jf c add/jf c edit or by hand-editing ~/.jfrog/jfrog-cli.conf.v6. Re-running jf login without the flag preserves whatever was previously saved.

Depends on jfrog/jfrog-cli-core#1600 (unmerged). go.mod currently has a temporary replace pinned to that PR's branch, marked for removal once it merges and a jfrog-cli-core release is cut. CI on this PR will fail until that dependency is resolved.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

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


@ehl-jf ehl-jf added the new feature Automatically generated release notes label Aug 6, 2026
Comment thread go.mod
// Should not be updated to 0.2.6 due to a bug (https://github.com/jfrog/jfrog-cli-core/pull/372)
github.com/c-bata/go-prompt => github.com/c-bata/go-prompt v0.2.5
// JGC-520: point at in-progress jfrog-cli-core branch - remove before merging
github.com/jfrog/jfrog-cli-core/v2 => github.com/jfrog/jfrog-cli-core/v2 v2.60.1-0.20260806050251-7359ffdda915

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please handle before merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature Automatically generated release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants