Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

jans-cli: Prompt for scopes optional parameter says array of string separated by _,» but the example is space separated #1350

Closed
ossdhaval opened this issue May 17, 2022 · 1 comment · Fixed by #2429
Closed

jans-cli: Prompt for scopes optional parameter says array of string separated by _,» but the example is space separated #1350

ossdhaval opened this issue May 17, 2022 · 1 comment · Fixed by #2429
Labels
comp-jans-cli-tui Component affected by issue or PR effort-1 Relative effort required for completion of issue or PR good first issue Good for newcomers help wanted Extra attention is needed kind-bug Issue or PR is a bug in existing functionality priority-5 Issue or PR addresses a very minor concern. Not planned for active development. Will not be fixed.
Milestone
1.0.3

Comments

@ossdhaval
Copy link
Contributor

How to reproduce:

  1. install Janssen server
  2. Invoke jans-cli from command line
  3. select option to register new OpenID connect client
  4. provide details for all mandatory parameters
  5. say yes to providing optional parameters
  6. select option to add value for scopes

Prompt for scopes optional parameter says array of string separated by _,» but the example is space separated

«Provide list of scopes granted to the client. Type: array of string separated by _,»
Example: read write dolphin

Originally posted by @ossdhaval in #1158 (comment)
@mo-auto mo-auto added the comp-jans-cli-tui Component affected by issue or PR label May 17, 2022
@ossdhaval ossdhaval added kind-bug Issue or PR is a bug in existing functionality good first issue Good for newcomers help wanted Extra attention is needed effort-1 Relative effort required for completion of issue or PR priority-5 Issue or PR addresses a very minor concern. Not planned for active development. Will not be fixed. labels May 17, 2022
@moabu moabu added this to the 1.0.1 milestone Jun 15, 2022
@moabu moabu modified the milestones: 1.0.1, 1.0.2 Jul 8, 2022
@moabu moabu modified the milestones: 1.0.2, 1.0.3 Aug 30, 2022
devrimyatar added a commit that referenced this issue Sep 20, 2022
@devrimyatar
fix: jans-cli seperation char (ref: #1350)
204daf8
@devrimyatar
Copy link
Contributor

image

@devrimyatar devrimyatar mentioned this issue Sep 20, 2022
Merged
yuriyz pushed a commit that referenced this issue Sep 28, 2022
@devrimyatar
fix: jans-cli fixes (#2429)
c9673dc 
* fix: jans-cli client clientName

* fix: jans-cli config-api pagination

* fix: jans-cli seperation char (ref: #1350)

* fix: jans-cli identify if patch property is array

* fix: jans-cli sort optional fields (ref: #1349)

* fix: jans-cli add log_dir to ini file and arg (ref: #1655)

* fix: jans-cli _b for escaping prompt (ref: #1657)

* feat: jans-cli tabulate UMA Resources

* fix: jans-cli array values for patch (ref: #1753)

* fix: jans-cli display last item in tabulated data

* fix: jans-cli item numbers in tabulated data

* fix: jans-cli hide user operations for openbanking (ref: #1878)

* fix: jans-cli hide fido2 for openbanking (ref: #1911)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
comp-jans-cli-tui Component affected by issue or PR effort-1 Relative effort required for completion of issue or PR good first issue Good for newcomers help wanted Extra attention is needed kind-bug Issue or PR is a bug in existing functionality priority-5 Issue or PR addresses a very minor concern. Not planned for active development. Will not be fixed.
Projects
None yet
Milestone
1.0.3
Development

Successfully merging a pull request may close this issue.

fix: jans-cli fixes JanssenProject/jans
4 participants
@ossdhaval @devrimyatar @moabu @mo-auto