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

PAYARA-1394 OAuth client integration #2695

Merged
merged 5 commits into from May 1, 2018

Conversation

Cousjava
Copy link
Contributor

@Cousjava Cousjava commented May 1, 2018

No description provided.

* PAYARA-1394 OAuth2 integration - new annotation and handler for it

* PAYARA-1394 Compiling version of basic mechanism

Does not work due to OSGi issues

* PAYARA-1394 resolved OSGi errors

* PAYARA-1394 removed HK2 annotations, using pure CDi

* PAYARA-1394 added import for annotation

* PAYARA-1394 oauth mechanism handler successfuly created, although innjection in it fails

* PAYARA-1394 Another way to produce the mechanism

Fails to inject OAuth2State

* PAYARA-1394 OAuth handler now works

* PAYARA-1394 Added default identity store

OAuth integration now appears to work, but needs further testing

* PAYARA-1394 authentication mechanism is now only used when a request is made to a secured page

Also split up OAuthState so some methods are part of the public api

* PAYARA-1394 OAuth integration now only is called when necessary

* PAYARA-1394 Split state into two files, added moduleinfo and javadoc comments

* PAYARA-1394 minor bug fixes, now  only use one logger name

* PAYARA-1394 Added enviroment variable substitution for the annotation

* PAYARA-1394 Correct scope name, remove extraenous code

* PAYARA-1394 add comma in imports; fix incorrect merge
@Cousjava Cousjava added this to the Payara 4.x milestone May 1, 2018
@Cousjava
Copy link
Contributor Author

Cousjava commented May 1, 2018

Jenkins, quick test Payara 4 please

@payara-ci
Copy link
Contributor

Quick build and test passed!

@Cousjava Cousjava requested a review from arjantijms May 1, 2018 09:27
@arjantijms arjantijms merged commit 17d3075 into payara:Payara4 May 1, 2018
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