imbajin commented on code in PR #2912:
URL:
https://github.com/apache/incubator-hugegraph/pull/2912#discussion_r2554945670
##########
hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/api/auth/ManagerAPI.java:
##########
@@ -73,7 +73,7 @@ public String createManager(@Context GraphManager manager,
AuthManager authManager = manager.authManager();
validUser(authManager, user);
- String creator = HugeGraphAuthProxy.getContext().user().username();
+ String creator = HugeGraphAuthProxy.username();
Review Comment:
⚠️ **Important: 全局一致性检查**
建议搜索整个代码库中所有 `HugeGraphAuthProxy.getContext().user().username()`
的调用点,确保都已替换为新的 `username()` 方法,避免遗漏其他潜在的 NPE 风险点。
可以使用以下命令检查:
```bash
grep -r "HugeGraphAuthProxy.getContext().user().username()"
--include="*.java"
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
