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.

Sign up for GitHub

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

Jump to bottom

URLPatternSpec.setURLPatternArray invalidates two prefix patterns that have a common prefix #56

Closed
fl4via opened this issue Oct 17, 2019 · 1 comment
Closed

URLPatternSpec.setURLPatternArray invalidates two prefix patterns that have a common prefix #56

fl4via opened this issue Oct 17, 2019 · 1 comment

Comments

@fl4via
Copy link

fl4via commented Oct 17, 2019 •
edited

If two patterns in the URL pattern array are of the prefix type (/blablabla/*) and are of the form:
/pattern/* and /patternblablabla/*, PatternSpec throws IllegalArgumentException indicating a prefix pattern in pattern list:
https://github.com/eclipse-ee4j/jacc/blob/master/api/src/main/java/javax/security/jacc/URLPatternSpec.java#L307
fl4via referenced this issue in fl4via/jacc Oct 17, 2019
@fl4via
Resolves eclipse-ee4j#56 At URLPatternSpec.setURLPatternArray, we nee…
0bbc1ef 
…d to verify if super.implies(urlPatternArray[i] to define if we have an invalid prefix pattern
fl4via referenced this issue in fl4via/jacc Oct 18, 2019
@fl4via
Resolves eclipse-ee4j#56 At URLPatternSpec.setURLPatternArray, we nee…
307167a 
…d to verify if super.implies(urlPatternArray[i] to define if we have an invalid prefix pattern

Signed-off-by: Flavia Rainone <frainone@redhat.com>
@fl4via fl4via mentioned this issue Oct 18, 2019
Closed
@fl4via
Copy link
Author

fl4via commented Oct 18, 2019

The problem that resulted in this issue is not in the spec implementation. So, this issue is invalid and for that reason I'm closing it.

@fl4via fl4via closed this as completed Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fl4via