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.

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

jans-cli :unable to create UMA resource with given mendatory properties #1752

Closed
manojs1978 opened this issue Jul 11, 2022 · 10 comments
Closed
Assignees
Milestone

Comments

@manojs1978
Copy link
Contributor

manojs1978 commented Jul 11, 2022

Describe the bug
jans-cli :unable to create UMA resource with given mendatory properties
getting attached error

To Reproduce
Steps to reproduce the behavior:

  1. install jans
  2. open jans-cli
  3. select 8 for OAuth
  4. select 2 for UMA
  5. select 2 to create new UMA resource
  6. add mendatory properties
    as below:
    image

Expected behavior
should able create UMA resource with given mendatory properties

Screenshots
image

Desktop (please complete the following information):

  • OS: ubuntu
  • Browser [e.g. chrome, safari]
  • Version 20.04

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context

UMA resource name and description are mendatory properties without that cant create UMA resource

@devrimyatar
Copy link
Contributor

@pujavs
name and description are should be required, as error given by jans-config-api

@devrimyatar
Copy link
Contributor

If name and description is provided, it creates
image

@pujavs
Copy link
Contributor

pujavs commented Jul 15, 2022

@devrimyatar @manojs1978 , fixed Mandatory fields via #1805

image

@pujavs pujavs assigned manojs1978 and unassigned pujavs Jul 18, 2022
@pujavs
Copy link
Contributor

pujavs commented Jul 18, 2022

Assigning it to @manojs1978 to test the fix

@moabu
Copy link
Member

moabu commented Jul 21, 2022

Hey @manojs1978,

So you have a status update on this ?

@manojs1978
Copy link
Contributor Author

Hi @moabu ,
i ll check with latest build and update this ticket

@manojs1978
Copy link
Contributor Author

manojs1978 commented Jul 27, 2022

verified on latest build
i was not able to reach to that step ,getting below error when added description .
@pujavs
image

below is error in error log:

Traceback (most recent call last):
File "/opt/jans/jans-cli/config-cli.py", line 2273, in main
cli_object.runApp()
File "/opt/jans/jans-cli/config-cli.py", line 2260, in runApp
self.display_menu(self.menu)
File "/opt/jans/jans-cli/config-cli.py", line 1854, in display_menu
self.display_menu(menu.get_child(menu_numbering[int(selection)]))
File "/opt/jans/jans-cli/config-cli.py", line 1854, in display_menu
self.display_menu(menu.get_child(menu_numbering[int(selection)]))
File "/opt/jans/jans-cli/config-cli.py", line 1857, in display_menu
getattr(self, 'process_' + m.method)(m)
File "/opt/jans/jans-cli/config-cli.py", line 1203, in process_get
self.display_menu(endpoint.parent)
File "/opt/jans/jans-cli/config-cli.py", line 1852, in display_menu
self.display_menu(menu.parent)
File "/opt/jans/jans-cli/config-cli.py", line 1854, in display_menu
self.display_menu(menu.get_child(menu_numbering[int(selection)]))
File "/opt/jans/jans-cli/config-cli.py", line 1857, in display_menu
getattr(self, 'process_' + m.method)(m)
File "/opt/jans/jans-cli/config-cli.py", line 1203, in process_get
self.display_menu(endpoint.parent)
File "/opt/jans/jans-cli/config-cli.py", line 1857, in display_menu
getattr(self, 'process_' + m.method)(m)
File "/opt/jans/jans-cli/config-cli.py", line 1452, in process_post
model = self.get_input_for_schema_(schema, model_class, required_only=True)
File "/opt/jans/jans-cli/config-cli.py", line 1412, in get_input_for_schema_
modelObject = model(**data)
File "/opt/jans/jans-cli/jca/swagger_client/models/uma_resource.py", line 91, in init
self.id = id
File "/opt/jans/jans-cli/jca/swagger_client/models/uma_resource.py", line 182, in id
raise ValueError("Invalid value for id, must not be None") # noqa: E501
ValueError: Invalid value for id, must not be None
~

@manojs1978 manojs1978 assigned pujavs and unassigned manojs1978 Aug 3, 2022
@moabu moabu modified the milestones: 1.0.2, 1.0.3 Aug 30, 2022
@pujavs
Copy link
Contributor

pujavs commented Sep 9, 2022

I had tested this earlier as well and communicated via DM that am not able to replicate. I have retested again today. and still not able to replicate. Request you to please see attached video and confirm if you are still facing issue.
Note i have tested on LDAP setup.[Uma_Resource_Creation_09Sep2022.zip]n(https://github.com/JanssenProject/jans/files/9536782/Uma_Resource_Creation_09Sep2022.zip)

@manojs1978
Copy link
Contributor Author

ok
checking

@manojs1978
Copy link
Contributor Author

issue is not reproducible in latest build
1752_latestbuild

actually i have retested long back in july month but now its working fine with latest build.

thank you.

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

No branches or pull requests

4 participants