javaee-samples / javaee7-samples

Java EE 7 Samples
https://travis-ci.org/javaee-samples/javaee7-samples
Other
2.51k stars 1.66k forks source link

JASPIC ServerAuthModule and ServerAuthContext spec compliance fixes #375

Open stoty opened 7 years ago

stoty commented 7 years ago

This contains two sets of fixes:

The first fix is important, as currently the tests fail to return proper http status codes for unathenticated protected resources on multiple app servers.

The second fix is just for complying with the letter of the spec, as the requestPolicy is not actually used in any of the current tests.

arjantijms commented 7 years ago

I've glanced over the PR and it looks good at first sight, but I'm very busy now with JSF 2.3 (final day before it closes). But haven't forgotten about this PR ;) Will look at it soon in more detail.

Thanks for now!