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

Adding new features and refactoring #52

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

uPagge
Copy link

@uPagge uPagge commented Nov 3, 2020

Hi, first of all thank you very much for this project, it helped a lot on our project :)

But unfortunately, everything was not as smooth as we would like, so the plugin had to be updated. I open the pull request because I hope that these are useful changes.

For reference, we use our own oauth2 server in our project

Refactoring
This mainly affected the creation and updating of entities and groups. I Packed everything in a DTO class for convenience.
I also expanded the number of fields that can be obtained from the SSO server

New features
There is a new field in the settings, the user exit token. Why is it needed? We needed this when it was necessary to make a simultaneous exit in all SSO-related applications. In other words, if a user logs out of app A, then they must also log out of app B, which is lifray.

In order for the SSO server to log out any user to liferay, it calls the /portal/logout method with the ServiceAuth header, which is set in the settings, and the request parameter of the user's UUID in liferay

For more information about the implementation, see LibFilter#processFilter.

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.

1 participant