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

UP-4685 Adds ServerProfileMapper #668

Merged

Conversation

vertein
Copy link
Contributor

@vertein vertein commented Jun 9, 2016

Allows the profile of a user to be dictated by what server the user is coming from, so that the user can test different profiles based on server.

Came up for us when one of our system schools (UW-River Falls) wanted to test our new profile via a specific url (my.uwrf.edu) but still retain the current profile at the old host (my.wisconsin.edu).

Adds the ability to select servers based on regular expression.

This is turned off by default, but does include commented examples of how to use.

Matcher matcher = this.pattern.matcher(userServerName);

if(matcher.matches()){
logger.debug("User {} will be given {} profile because server {} matches regular expression {}",
Copy link
Contributor

Choose a reason for hiding this comment

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

This message is clear & awesome.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks!

@drewwills
Copy link
Contributor

👍 Looks very good.

@timlevett
Copy link
Contributor

👍

@vertein vertein merged commit 18c2dd0 into uPortal-Project:master Jun 13, 2016
@vertein vertein deleted the UP-4685-AddsServerProfileMapper branch June 13, 2016 12:50
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

Successfully merging this pull request may close these issues.

None yet

3 participants