Skip to content

Conversation

@shwstppr
Copy link
Contributor

@shwstppr shwstppr commented Sep 21, 2021

Description

Fixes #5477
Fixes #5478

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

Before change:

Screenshot from 2021-09-21 12-19-12

Screenshot from 2021-09-21 12-19-36

After change:
Screenshot from 2021-09-21 12-22-38

Screenshot from 2021-09-21 12-22-59

How Has This Been Tested?

Fixes apache#5478

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@rohityadavcloud
Copy link
Member

@blueorangutan ui

@rohityadavcloud rohityadavcloud added this to the 4.16.0.0 milestone Sep 21, 2021
@rohityadavcloud
Copy link
Member

@blueorangutan ui

@blueorangutan
Copy link

@rhtyd a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5481 (SL-JID-673)

@rohityadavcloud
Copy link
Member

rohityadavcloud commented Sep 21, 2021

@shwstppr couple of issues, not sure if they are due to the qa.cloudstack.cloud backend of 4.15.1: (the qa.cloudstack.cloud is SAMl enabled and can be used for testing)

  1. Check if the normal user/role type should be shown this form, it fails:
    Screenshot from 2021-09-21 12-25-37

  2. On creating user as normal user type, that fails too (in its own account):
    Screenshot from 2021-09-21 12-26-14

  3. As domain admin, the IDP list was somehow not shown in the create user form:
    Screenshot from 2021-09-21 12-27-16

  4. As root admin, I created a new user 'rick' and while creating the user ticked authorise saml sso and selected the IDP however the newly-created user is not SAML2 type:
    Screenshot from 2021-09-21 12-30-45

  5. For an already authorised saml user, when clicking edit saml authorisation button the IDP is pre-selected as 0 (something wrong in option/value key?)
    Screenshot from 2021-09-21 12-36-20

Note: this form also seems to pass invalid entityid in the API, so the user isn't really SAML enabled.

@shwstppr please also test/fix any issues with the createAccount form (which also has the saml authorisation option).

Fixes apache#5477

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr shwstppr changed the title ui: fix create user with saml ui: fix create account/user with saml Sep 21, 2021
@shwstppr
Copy link
Contributor Author

@rhtyd Verified mentioned issues with PR build:

  1. Normal User role cannot create new users in their own account. Old UI doesn't provide the option and option wasn't there in this PR build as well.
  2. Same as 1
  3. Fixed this case
  4. Fixed with this PR
  5. Didn't see this behaviour with this PR build.

Addressed #5477 and #5478
@blueorangutan ui

@blueorangutan
Copy link

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr
Copy link
Contributor Author

@blueorangutan ui

@shwstppr shwstppr requested a review from Pearl1594 September 21, 2021 07:55
@blueorangutan
Copy link

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@shwstppr shwstppr requested review from Pearl1594, rohityadavcloud and utchoang and removed request for Pearl1594 September 21, 2021 07:55
@blueorangutan
Copy link

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5481 (SL-JID-674)

@blueorangutan
Copy link

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5481 (SL-JID-675)

Copy link

@utchoang utchoang left a comment

Choose a reason for hiding this comment

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

LGTM!

@rohityadavcloud
Copy link
Member

@blueorangutan ui

Copy link
Member

@rohityadavcloud rohityadavcloud left a comment

Choose a reason for hiding this comment

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

LGTM - I'll test and confirm shortly

@rohityadavcloud
Copy link
Member

@blueorangutan ui

@blueorangutan
Copy link

@rhtyd a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5481 (SL-JID-676)

@rohityadavcloud
Copy link
Member

rohityadavcloud commented Sep 21, 2021

@shwstppr outstanding/new issues:

  1. Minor - does not refresh user detail page on edit/saving the authorise saml sso form
  2. As domain admin, the entity ID is not updating - still 0? And on saving the form still passes 0 as entityid
  3. Minor - Create user form doesn't work for domain user the #/user path (as domainid is not passed)

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@rohityadavcloud
Copy link
Member

@shwstppr is this ready for final test/merge, or you addressing minor issues too?
@blueorangutan ui

@blueorangutan
Copy link

@rhtyd a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5481 (SL-JID-678)

@shwstppr
Copy link
Contributor Author

shwstppr commented Sep 21, 2021

@rhtyd let's move ahead with one for 2

As domain admin, the entity ID is not updating - still 0? And on saving the form still passes 0 as entityid

I'll fix 1 and 3 in another PR #5487

@rohityadavcloud
Copy link
Member

LGTM tested!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problem adding SAML user into a existing Account Problem adding SAML user as Domain Admin

4 participants