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

WELD-2517: Weld SE - fix SecurityManager-related issues. #1895

Merged
merged 1 commit into from Jan 31, 2019

Conversation

Projects
None yet
3 participants
@mkouba
Copy link
Member

commented Jan 28, 2019

No description provided.

@mkouba mkouba requested a review from manovotn Jan 29, 2019

@manovotn

This comment has been minimized.

Copy link
Contributor

commented Jan 29, 2019

We don't really have full SM testsuite but I think it'd be fitting to verify this the way we discovered this in the first place. @nziakova that should be the SE SM example, right?

@mkouba

This comment has been minimized.

Copy link
Member Author

commented Jan 29, 2019

Yep, that's what I did - run the numberguess SE example with SM manually.

@nziakova

This comment has been minimized.

Copy link
Collaborator

commented Jan 29, 2019

Yes, the se/numberguess one. Works fine now, thanks!

@manovotn manovotn merged commit a7d7f69 into weld:master Jan 31, 2019

5 checks passed

Weld CI - JDK 11 Build finished.
Details
Weld CI - JDK 8 Build finished.
Details
Weld Publish CI - JDK 11 Finished
Details
Weld Publish CI - JDK 8 Finished
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.