Skip to content

Fix: add more config validation logic to networkRBAC plugin (#629)#897

Closed
KariHall619 wants to merge 1 commit into
mosn:mainfrom
KariHall619:main
Closed

Fix: add more config validation logic to networkRBAC plugin (#629)#897
KariHall619 wants to merge 1 commit into
mosn:mainfrom
KariHall619:main

Conversation

@KariHall619
Copy link
Copy Markdown

Fix #629

This pull request adds additional configuration validation to the networkRBAC plugin.

What’s changed:

  • Added validation for matcher: exactMatchMap `

Test:

  • Added unit tests in config_test.go to verify invalid configurations are caught
  • All tests passed via go test ./...

Please let me know if any adjustments are needed. Thanks!

@github-actions github-actions Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add more validation to the conf in networkRBAC plugin

1 participant