-
Notifications
You must be signed in to change notification settings - Fork 824
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
Move IdP Alias Handling to separate Class #2737
Move IdP Alias Handling to separate Class #2737
Conversation
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/187079914 The labels on this github issue will be updated when the story is started. |
&& existingEntity.getAliasZid().equals(requestBody.getAliasZid()); | ||
} | ||
|
||
// alias ID must not be set when no alias existed already |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
caller must not choose alias ID, instead new alias ID will be generated
void setAliasId(String aliasId); | ||
|
||
@Nullable | ||
String getAliasZid(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose that aliasZid is aliasZoneId, right? Might be better to either document it (javadoc) or rename the property to something clearer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added Javadoc comments in commit 0197893
see issue #2505
Move the handling of alias IdPs to a separate class so that it can be re-used for the future alias feature for SCIM users.