Skip to content

Commit 8494366

Browse files
committed
fix: add security comment for urlopen usage
1 parent 9f439e9 commit 8494366

File tree

1 file changed

+1
-0
lines changed
  • src/datapilot/core/knowledge

1 file changed

+1
-0
lines changed

src/datapilot/core/knowledge/cli.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ def handle_knowledge_base(self, public_id):
6767
req = Request(url, headers=headers)
6868

6969
try:
70+
# URL scheme validated above - only HTTP/HTTPS allowed
7071
with urlopen(req, timeout=30) as response: # noqa: S310
7172
data = response.read()
7273
self.send_response(200)

0 commit comments

Comments
 (0)