Skip to content

Explicitly requires the root group to be the VO name - #112

Open
enricovianello wants to merge 1 commit into
mainfrom
add-root-validation
Open

enricovianello wants to merge 1 commit into
mainfrom
add-root-validation

Conversation

@enricovianello

@enricovianello enricovianello commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Enforce the requirement from GFD.182 §3.4.1.2 that every FQAN’s root group matches the VO name declared in the attribute certificate.

Both validateAC() overloads now reject the entire AC if any FQAN is inconsistent, reporting the dedicated fqanDoesntMatchVo error without filtering or reordering attributes.

Adds regression tests using signed ACs, covering valid groups and roles, VO mismatches, prefix collisions, case sensitivity, and mixed-VO FQAN lists.

Fixes #42.

Comment thread src/main/java/org/italiangrid/voms/ac/impl/DefaultVOMSValidationStrategy.java Outdated
Comment thread src/main/java/org/italiangrid/voms/ac/impl/DefaultVOMSValidationStrategy.java Outdated
@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 2026

Copy link
Copy Markdown

This branch has not been deployed

No deployments
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

Successfully merging this pull request may close these issues.

Missing check that top-level group is the VO name

2 participants