Skip to content

CLI - Hide the --server-issued-login param#4905

Merged
bfops merged 2 commits intomasterfrom
bfops/hide-login
Apr 27, 2026
Merged

CLI - Hide the --server-issued-login param#4905
bfops merged 2 commits intomasterfrom
bfops/hide-login

Conversation

@bfops
Copy link
Copy Markdown
Collaborator

@bfops bfops commented Apr 27, 2026

Description of Changes

Users have an odd tendency to overuse this param, which is really meant for testing and some exceptional circumstances.

This PR hides it from the help so that it is less discoverable.

API and ABI breaking changes

None.

Expected complexity level and risk

1

Testing

  • Helptext no longer includes --server-issued-login

Copy link
Copy Markdown
Collaborator

@jdetter jdetter left a comment

Choose a reason for hiding this comment

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

LGTM - not tested by me but this one is trivial

@bfops bfops enabled auto-merge April 27, 2026 15:56
@bfops bfops added this pull request to the merge queue Apr 27, 2026
Merged via the queue into master with commit 2e01ce8 Apr 27, 2026
52 of 54 checks passed
@bfops bfops deleted the bfops/hide-login branch April 28, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants