imbajin commented on code in PR #2966:
URL: https://github.com/apache/hugegraph/pull/2966#discussion_r2911691470
##########
hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/api/space/GraphSpaceAPI.java:
##########
@@ -76,6 +76,7 @@ public class GraphSpaceAPI extends API {
@Produces(APPLICATION_JSON_WITH_CHARSET)
public Object list(@Context GraphManager manager,
@Context SecurityContext sc) {
+ checkPdModeEnabled(manager);
Review Comment:
‼️ `checkPdModeEnabled()` is added to the other GraphSpace endpoints, but
`/graphspaces/profile` is still unguarded. In standalone mode this method will
continue to return a different result instead of the new 400, so the API
contract remains inconsistent. Please add the same guard to `listProfile()` and
extend the standalone coverage to hit `/graphspaces/profile` as well.
--
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]
