youngkermit8-coder opened a new pull request, #1637: URL: https://github.com/apache/rocketmq-dashboard/pull/1637
## Summary - explicitly persist SQL `NULL` when an existing plain-access IP whitelist is cleared - keep the narrower fix inside the transactional plain-access upsert path - add a repository regression test that inspects the generated nullable column assignment ## Root cause `createAndUpdatePlainAccessConfig` normalized an empty `whiteRemoteAddress` to `null` and then called MyBatis-Plus `updateById`. MyBatis-Plus skips null entity fields by default, so the old `white_remote_address` remained stored even though the update succeeded. The fix uses an explicit update assignment only when this field is being cleared. It intentionally does not change the entity-wide update strategy, so unrelated ACL user updates continue preserving an existing whitelist. ## Validation - regression test failed on the unmodified implementation, then passed with this change - `MybatisPlusAclRepositoryTest,AclServiceTest,AclControllerTest`: 61/61 passed - Checkstyle: 0 violations - `mvn package -DskipTests`: passed - full backend suite: 903/905 passed; the two `ToolGatewayServiceTest` JSON Schema failures reproduce unchanged on clean base `ffaff10` Closes #1634 -- 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]
