Aias00 opened a new pull request, #6384: URL: https://github.com/apache/shenyu/pull/6384
<!-- Describe your PR here; e.g. Fixes #issueNo --> This PR adds the missing `system:plugin:modify` permission check to the single namespace plugin sync endpoint. The bulk plugin sync endpoint already requires `system:plugin:modify`; the single-plugin sync path can publish plugin data, including uploaded plugin JAR content, to connected gateway nodes and should enforce the same permission boundary. Make sure that: - [ ] You have read the [contribution guidelines](https://shenyu.apache.org/community/contributor-guide). - [x] You submit test cases (unit or integration tests) that back your changes. - [ ] Your local test passed `./mvnw clean install -Dmaven.javadoc.skip=true`. Tested: - `./mvnw -pl shenyu-admin -Dskip.checkstyle=true -DskipLicense=true -Dspotless.check.skip=true -Djacoco.skip=true -Dtest=NamespacePluginControllerTest test -q` - `./mvnw -pl shenyu-admin -Dskip.checkstyle=true -DskipLicense=true -Dspotless.check.skip=true -Djacoco.skip=true -Dtest=NamespacePluginControllerTest,DataPermissionControllerTest,SelectorControllerTest test -q` - `./mvnw -pl shenyu-admin -Dskip.checkstyle=true -DskipLicense=true -Dspotless.check.skip=true -Djacoco.skip=true test -q` - `./mvnw -pl shenyu-admin -DskipTests -Djacoco.skip=true verify -q` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
