Actions
Bug #30720
closedhammer auth login oauth breaks convention by using dash as word separator in option value
Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Difficulty:
Triaged:
Yes
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:
Description
hammer auth login oauth is the only place in hammer where option value uses dash as word separator, instead of underscore.
$ hammer auth login oauth --help Usage: hammer auth login oauth [OPTIONS] Options: -a, --oidc-authorization-endpoint OPENIDC-AUTHORIZATION-ENDPOINT Openidc provider URL which issues authentication code (two factor only) -c, --oidc-client-id OPENIDC-CLIENT-ID Client id used in the Openidc provider -f, --two-factor Authenticate with two factor -h, --help Print help -p, --password PASSWORD Password to access the remote system -r, --oidc-redirect-uri OPENIDC-REDIRECT-URI Redirect URI for the authentication code grant flow -t, --oidc-token-endpoint OPENIDC-TOKEN-ENDPOINT Openidc provider URL which issues access token -u, --username USERNAME Username to access the remote system
Compare with e.g. hammer report-template schedule
(some lines skipped)
$ hammer report-template schedule --help Usage: hammer report-template schedule [OPTIONS] Options: --generate-at GENERATE_AT UTC time to generate report at --gzip GZIP Compress the report using gzip One of true/false, yes/no, 1/0. --mail-to MAIL_TO If set, scheduled report will be delivered via e-mail. Use ',' to Separate multiple email addresses. --path PATH Path to directory where downloaded content will be saved. Only usable if wait is specified --report-format REPORT_FORMAT Report format, defaults to 'csv'
To be specific, OPENIDC-AUTHORIZATION-ENDPOINT
vs GENERATE_AT
- it should be OPENIDC_AUTHORIZATION_ENDPOINT
Updated by The Foreman Bot about 4 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/536 added
Updated by The Foreman Bot about 4 years ago
- Fixed in Releases hammer-cli-foreman-2.3.0 added
Updated by Anonymous about 4 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset hammer-cli-foreman|8514b5db9c0e5b455edfa20277b0356f244ddaca.
Updated by The Foreman Bot about 4 years ago
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/538 added
Updated by Oleh Fedorenko about 4 years ago
- Fixed in Releases hammer-cli-foreman-2.1.3 added
Actions