You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hugegraph.apache.org by ji...@apache.org on 2026/05/31 04:07:28 UTC
(hugegraph-ai) 02/02: docs(code-scan): sync fixed issue statuses
This is an automated email from the ASF dual-hosted git repository.
imbajin pushed a commit to branch goal-scan
in repository https://gitbox.apache.org/repos/asf/hugegraph-ai.git
commit 53f3ebac1717e589f57d3cbb979e86e75a8d5b31
Author: imbajin <ji...@apache.org>
AuthorDate: Sun May 31 11:59:32 2026 +0800
docs(code-scan): sync fixed issue statuses
- correct status rows for completed requested fixes
- leave unrelated open scan findings marked open
- keep CS-001, CS-002, CS-004, CS-008, CS-011, CS-012, CS-013, and CS-030 fixed
---
.workflow/code-scan/reports/issues.md | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/.workflow/code-scan/reports/issues.md b/.workflow/code-scan/reports/issues.md
index 665de54d..6c8277f5 100644
--- a/.workflow/code-scan/reports/issues.md
+++ b/.workflow/code-scan/reports/issues.md
@@ -45,7 +45,7 @@ Issue IDs use `CS-NNN`. Priorities are P0 highest through P5 lowest.
- Module: `hugegraph-python-client`
- Layer: transport / routing
- Paths: `hugegraph-python-client/src/pyhugegraph/utils/huge_requests.py:100`, `hugegraph-python-client/src/pyhugegraph/utils/huge_requests.py:120`, `hugegraph-python-client/src/pyhugegraph/api/auth.py:37`, `hugegraph-python-client/src/pyhugegraph/api/auth.py:84`
-- Status: fixed
+- Status: open
- Evidence: `HGraphSession.resolve()` passes leading-slash paths to `urljoin`, which replaces the base URL path. A base like `http://host/proxy` plus `/auth/groups` resolves to `http://host/auth/groups`.
- Impact: HugeGraph behind path-prefix proxies or ingress routes is addressed incorrectly.
- Recommendation: Treat route strings as API-root-relative paths inside the client, or reject base URLs with path prefixes explicitly.
@@ -69,7 +69,7 @@ Issue IDs use `CS-NNN`. Priorities are P0 highest through P5 lowest.
- Module: `hugegraph-python-client`
- Layer: transport / routing
- Paths: `hugegraph-python-client/src/pyhugegraph/utils/huge_config.py:44`, `hugegraph-python-client/src/pyhugegraph/utils/huge_config.py:73`, `hugegraph-python-client/src/pyhugegraph/utils/huge_router.py:139`
-- Status: fixed
+- Status: open
- Evidence: `HGraphConfig.__post_init__()` probes `/versions` with a fixed `0.5s` timeout. Most probe failures silently set `gs_supported=False`, making graphspace auth routes fail locally with `ValueError`.
- Impact: HugeGraph 1.7+ auth behavior becomes sensitive to a single transient version probe.
- Recommendation: Make graphspace configuration explicit or retry/observe the capability probe using the configured timeout.
@@ -81,7 +81,7 @@ Issue IDs use `CS-NNN`. Priorities are P0 highest through P5 lowest.
- Module: `hugegraph-python-client`
- Layer: client-api
- Paths: `hugegraph-python-client/src/pyhugegraph/api/graph.py:68`, `hugegraph-python-client/src/pyhugegraph/api/graph.py:73`, `hugegraph-python-client/src/pyhugegraph/api/graph.py:173`, `hugegraph-python-client/src/pyhugegraph/utils/huge_requests.py:144`
-- Status: fixed
+- Status: open
- Evidence: `getVertexByPage()`, `getVertexByCondition()`, and `getEdgeByPage()` concatenate labels and JSON properties directly into the URL.
- Impact: Property values containing `&`, `?`, `#`, `=`, or spaces can corrupt filters and query different data.
- Recommendation: Use `params=` or `urllib.parse.urlencode()` for all query parameters.
@@ -105,7 +105,7 @@ Issue IDs use `CS-NNN`. Priorities are P0 highest through P5 lowest.
- Module: `hugegraph-python-client`
- Layer: gremlin / error contract
- Paths: `hugegraph-python-client/src/pyhugegraph/api/gremlin.py:48`, `hugegraph-python-client/src/pyhugegraph/api/gremlin.py:54`, `hugegraph-python-client/src/tests/api/test_gremlin.py:99`
-- Status: open
+- Status: fixed
- Evidence: `GremlinManager.exec()` catches every exception and raises `NotFoundError`.
- Impact: Auth failures, transport errors, server errors, and syntax errors are indistinguishable upstream.
- Fix: `GremlinManager.exec()` no longer wraps all lower-level exceptions as `NotFoundError`; typed auth/transport/server/parse exceptions propagate from the transport boundary.
@@ -141,7 +141,7 @@ Issue IDs use `CS-NNN`. Priorities are P0 highest through P5 lowest.
- Module: `hugegraph-llm`
- Layer: API / models / config
- Paths: `hugegraph-llm/src/hugegraph_llm/api/models/rag_requests.py:27`, `hugegraph-llm/src/hugegraph_llm/api/rag_api.py:92`
-- Status: open
+- Status: fixed
- Evidence: `GraphConfigRequest` has concrete defaults and `set_graph_config()` writes every field to global `huge_settings` whenever `client_config` exists.
- Impact: A request intended to override one field can clear credentials, graph name, or graphspace for the whole process.
- Fix: request-scoped graph config updates now use `model_fields_set`, so only explicitly provided nested fields mutate `huge_settings`.
@@ -165,7 +165,7 @@ Issue IDs use `CS-NNN`. Priorities are P0 highest through P5 lowest.
- Module: `hugegraph-llm`
- Layer: LLM operator / graph import
- Paths: `hugegraph-llm/src/hugegraph_llm/operators/llm_op/property_graph_extract.py:53`, `hugegraph-llm/src/hugegraph_llm/operators/hugegraph_op/commit_to_hugegraph.py:134`, `hugegraph-llm/src/hugegraph_llm/flows/import_graph_data.py:65`
-- Status: open
+- Status: fixed
- Evidence: `filter_item()` converts non-string properties to strings, while `Commit2Graph` enforces schema Python types and skips mismatches. Edge writes can continue and the flow reports success.
- Impact: KG import can partially fail while the UI reports success.
- Fix: `filter_item()` no longer stringifies non-string property values, so extracted JSON numeric values survive into `Commit2Graph` schema validation.