Bug 1451891 - Auth External Auth - AD - SSUI, Users groups not being evaluated properly
Summary: Auth External Auth - AD - SSUI, Users groups not being evaluated properly
Keywords:
Status: CLOSED WORKSFORME
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - Service
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: GA
: cfme-future
Assignee: Chris Hale
QA Contact: Matt Pusateri
URL:
Whiteboard: auth:externalauth:ad:ssui
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-05-17 18:57 UTC by Matt Pusateri
Modified: 2017-07-11 13:48 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-06-29 21:21:44 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Comment 2 Joe Vlcek 2017-05-17 20:27:11 UTC
Matt,

Can you PM me the creds were you are seeing this issue.

Along with the creds for you AD.

Thank you, JoeV

Comment 3 Matt Pusateri 2017-05-17 20:31:55 UTC
So test-user8 is not a member of user-group-ad which according to the logs is being evaluated.  He's only a member of evmgroup-user

So groups are not being retrieved or evaluated properly.

Comment 6 Joe Vlcek 2017-05-17 20:48:29 UTC
Works for me.
I just successfully logged into <your appliance>/ui/service as test-user8

Comment 7 Matt Pusateri 2017-05-18 15:58:35 UTC
So in 5.7.2.1, test-user8 can't log into classic or SSUI, I'm guessing part of this is a problem with the users properties.  Though that doesn't explain evaluating the wrong group.

Comment 8 Matt Pusateri 2017-05-18 16:07:45 UTC
I was just able to reproduce this on 5.8.0-14-rc3, the same box as above. I think the steps to reproduce are to log in with user who doesn't have perms. Then log in with a user who does

Login with test-user5 same pwd as test-user8, test-user5 fails due to evmgroup-operator not have SSUI perms. Then test-user8 fails to login and has test-user5's group user-group-ad evaluated.

Comment 9 Matt Pusateri 2017-05-18 17:14:39 UTC
So I changed the title on this, test-user8 shouldn't be able to log in as evmgroup-user has no SSUI perms, But that authorization doesn't happen, as the wrong group is getting evaluated.

Also I would argue that evmgroup-user should be allowed to log into SSU (RFE bug: https://bugzilla.redhat.com/show_bug.cgi?id=1452320)

Comment 11 Chris Kacerguis 2017-06-01 21:00:14 UTC
Chris - Can you please take a look?

Comment 12 Matt Pusateri 2017-06-02 13:41:04 UTC
I think it is happening in prod, it just doesn't prevent users from logging in b/c SSUI has worked around it.  I think it should be high, b/c it critical the way the API handles logins and is the basis for security.  If the API is authorizing when it should be, that's a serious concern!

Comment 13 Chris Hale 2017-06-05 20:05:09 UTC
Tried to take a look at this.  Tried https://10.8.199.225/ and the url didnt come up.  Is there an environment I can look at this happening?

Comment 14 Matt Pusateri 2017-06-05 20:09:14 UTC
It'll have to be recreated, all our QE appliances are temporary.

Comment 15 Chris Hale 2017-06-13 19:53:11 UTC
Matt. Let me know if you get a appliance set up that this can be tested on.

Comment 17 Chris Hale 2017-06-13 20:06:04 UTC
I do not have an AD setup.  It would be a lot easier if we had a environment or AD env set up that I could connect to in order to help diagnose .

Comment 18 Matt Pusateri 2017-06-13 20:13:12 UTC
Talk with your admins, there is most certainly a development AD environment for you to test against.

Comment 19 Chris Kacerguis 2017-06-13 21:18:27 UTC
Setting priority / severity to "unspecified" to allow for this BZ to go back through triage.

Comment 20 Matt Pusateri 2017-06-14 12:11:43 UTC
Part of the reason it may not be hit in Prod, is that the SSUI team worked around this in code.  If the backend is not evaluating authorization properly, I would argue that it is a high priority.

Comment 21 Chris Kacerguis 2017-06-14 21:36:30 UTC
A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/147234269

Comment 23 Chris Kacerguis 2017-06-29 21:21:59 UTC
Chris Kacerguis deleted the linked story in Pivotal Tracker


Note You need to log in before you can comment on or make changes to this bug.