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

TCK Challenge: ServletProfileSPITest#CheckMsgInfoKey #219

Open
markt-asf opened this issue Sep 19, 2024 · 1 comment
Open

TCK Challenge: ServletProfileSPITest#CheckMsgInfoKey #219

markt-asf opened this issue Sep 19, 2024 · 1 comment

Comments

@markt-asf
Copy link
Contributor

There is one definite issue and one possible issue with this test.

  1. Hard-coded requirement for Jakarta Authorization support.
boolean bIs115Compatible = false;

// ONLY if we are Jakarta Authorization compatible, we want to make an additional key test
if (bIs115Compatible) {
...

That boolean value needs to be externally configurable.

Once that has been addressed, there is a possible second issue with the alternative code branch

  1. The CallerPrincipalCallback check expects the user name to be set to j2ee but the user name isn't set until PasswordValidationCallback is called which happens after CallerPrincipalCallback. I do not see how the test can pass in its current form.

It is quite possible that the second issue is a problem with my understanding rather than the TCK. If, so any pointers to where I am going wrong would be appreciated.

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

No branches or pull requests

2 participants
@markt-asf and others