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 1925 jacc per app #2651

Merged
merged 7 commits into from Apr 24, 2018
Merged

Payara 1925 jacc per app #2651

merged 7 commits into from Apr 24, 2018

Conversation

arjantijms
Copy link
Contributor

No description provided.

@arjantijms
Copy link
Contributor Author

Note that main functional changes are in:

@ggam
Copy link

ggam commented Apr 18, 2018

Great to see this at last!

@arjantijms
Copy link
Contributor Author

@ggam

Great to see this at last!

Thanks! Very happy to have been able to implement this too. It's approximately the approach we discussed all that time back in the JACC group with Ron. If the JACC spec starts again I'd like to present this work as the POC to potentially base the spec revision on.

Copy link
Contributor

@MattGill98 MattGill98 left a comment

Choose a reason for hiding this comment

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

All looks good, just some semantic comments!

@@ -2606,6 +2609,14 @@ public boolean isJaxrsRolesAllowedEnabled() {
public void setJaxrsRolesAllowedEnabled(boolean jaxrsRolesAllowedEnabled) {
this.jaxrsRolesAllowedEnabled = jaxrsRolesAllowedEnabled;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Can these have an @OverRide?

private void createPolicyContextDirectory() {

String contextDirectoryName = getContextDirectoryName();
File d = new File(contextDirectoryName);
File contectDirectory = new File(contextDirectoryName);
Copy link
Contributor

Choose a reason for hiding this comment

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

gahhhh

@mulderbaba mulderbaba added this to the Payara 5.182 milestone Apr 19, 2018
@arjantijms
Copy link
Contributor Author

jenkins test

@payara payara deleted a comment from payara-ci Apr 24, 2018
@payara payara deleted a comment from payara-ci Apr 24, 2018
@payara payara deleted a comment from lprimak Apr 24, 2018
@payara payara deleted a comment from payara-ci Apr 24, 2018
@payara payara deleted a comment from payara-ci Apr 24, 2018
@payara-ci
Copy link
Contributor

Quick build and test passed!

@arjantijms arjantijms merged commit 1c50e3f into payara:master Apr 24, 2018
@arjantijms arjantijms deleted the PAYARA-1925-JaccPerApp branch July 9, 2018 13:29
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

6 participants