jakartaee / jakartaee/security

Add test for injecting OpenID mechanism by qualifier

Open
#343 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
66
Forks
43
PR merge metrics
No merged PRs in 30d

Description

There's no test in the TCK to cover injecting OpenID mechanisms with `qualifiers` attribute.

The test for injecting multiple auth mechanisms into a custom handler in [AppCustomAuthenticationMechanismHandler2IT](https://github.com/jakartaee/security/blob/7233713f44dc5a58fcc82fcb8167e400fccedca7/tck/app-custom-authentication-mechanism-handler2/src/test/java/ee/jakarta/tck/security/test/AppCustomAuthenticationMechanismHandler2IT.java#L42) only covers basic mechanisms.

We found out that, although both WildFly and GlassFish preview versions, and the underlying Soteria implementation, all passed the TCK, they failed to inject a modified case with OpenID mechanisms instead of Basic mechanisms. It would be good to add a test similar to the existing one for Basic authentication also for OpenID. Or, even better, for all other specified mechanisms (OpenID, Form, CustomForm).

More information in: https://github.com/jakartaee/security/issues/342

Contributor guide

Open the contributing guide

Research direction

Start with tck/app-custom-authentication-mechanism-handler2/src/test/java/ee/jakarta/tck/security/test/AppCustomAuthenticationMechanismHandler2IT.java, especially the existing test for injecting multiple basic mechanisms into a custom handler. Add the analogous OpenID case using the qualifiers attribute, then run AppCustomAuthenticationMechanismHandler2IT. Done means the TCK covers and passes OpenID mechanism injection with qualifiers.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
authentication, testing
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.