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

Bug fixes in building ResourceAccessManager's securityFilter #74

Merged
merged 1 commit into from
Aug 13, 2024
Merged

Bug fixes in building ResourceAccessManager's securityFilter #74

merged 1 commit into from
Aug 13, 2024

Conversation

groldan
Copy link
Member

@groldan groldan commented Aug 13, 2024

Increase test coverage for CatalogSecurityFilterBuilder to 98.3% and fix bugs found when rules produce AccessSummarys with a mixture allow all or deny all workspaces, and concrete workspaces.

@see 52d4555

@groldan
Bug fixes in building ResourceAccessManager's securityFilter
1bee573 
Increase test coverage for `CatalogSecurityFilterBuilder` to 98.3% and
fix bugs found when rules produce `AccessSummary`s with a mixture allow all or
deny all workspaces, and concrete workspaces.

@see 52d4555
@groldan groldan added the plugin Issues related to the GeoServer plugin label Aug 13, 2024
@groldan groldan merged commit f72f6f8 into geoserver:main Aug 13, 2024
10 checks passed
@groldan groldan deleted the fix_security_filter branch August 13, 2024 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
plugin Issues related to the GeoServer plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@groldan