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

Atribute form missing SAML URI values #452

Closed
nynymike opened this issue Jan 23, 2017 · 3 comments
Closed

Atribute form missing SAML URI values #452

nynymike opened this issue Jan 23, 2017 · 3 comments
Assignees
@dmogn
Labels
bug exclude from release notes issues found during QA that should be excluded from release notes
Milestone
CE 3.0.0

Comments

@nynymike
Copy link
Contributor

Why can't the user enter SAML URI's for these attributes?

image
@nynymike nynymike added the bug label Jan 23, 2017
@nynymike nynymike added this to the CE 3.0.0 milestone Jan 23, 2017
@nynymike nynymike assigned shekhar16 and dmogn and unassigned shekhar16 Jan 23, 2017
@dmogn
Copy link
Contributor

dmogn commented Jan 23, 2017 •
edited
Loading

@nynymike There is some logic for show/hide URI fields:
https://github.com/GluuFederation/oxTrust/blob/master/server/src/main/java/org/gluu/oxtrust/action/UpdateAttributeAction.java#L373
Attribute Origin should be equal to application configuration personCustomObjectClass.

I can simply remove this rule or change "hide" state to "disabled" state for URI fields (more user friendly, I think).

dmogn added a commit that referenced this issue Jan 23, 2017
@dmogn
Atribute form missing SAML URI values #452 - fix
5b455e3
@dmogn
Copy link
Contributor

dmogn commented Jan 23, 2017

Mike Schwartz: remove it...
Mike Schwartz: It's up to the admin to configure attributes correctly.

@dmogn
Copy link
Contributor

dmogn commented Jan 23, 2017

Fixed.

@dmogn dmogn closed this as completed Jan 23, 2017
@dmogn dmogn added the exclude from release notes issues found during QA that should be excluded from release notes label Jan 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dmogn dmogn

Labels
bug exclude from release notes issues found during QA that should be excluded from release notes
Projects
None yet
Milestone
CE 3.0.0
Development

No branches or pull requests
3 participants
@nynymike @dmogn @shekhar16