[HARDENING] _validate_imports misses non-line-start imports — switch to AST-based validation
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 50/100
調査の方向性
Start with _validate_imports in maithili_dsl/cli.py, which matches imports by line prefix, and reproduce the miss with x = 1; import os. Read CONTRIBUTING.md, since changes to the sandbox require a Security Considerations section in the PR. Replace the line scan with an ast.parse() walk over ast.Import and ast.ImportFrom nodes, and add a SyntaxError path. Done when the semicolon, indented, and from ... import cases are rejected in tests/test_security.py and existing tests still pass.
索引モデルが issue の本文から書いたものです。
説明
File: maithili_dsl/cli.py (_validate_imports)
The pre-exec import check is line-prefix-based (stripped.startswith('import ')), so statements not at the start of a line are missed:
x = 1; import os # passes _validate_imports (verified)
Runtime is still guarded by the __import__ wrapper, so this is not a sandbox escape — only whitelisted modules load either way — but it silently defeats the documented "raw Python imports are rejected before execution" layer, and prefix matching will also miss conditional/indented imports.
Fix: parse with ast.parse() and walk ast.Import / ast.ImportFrom nodes instead of scanning lines. Bonus: this also gives precise line numbers for error messages, and a clean SyntaxError path for malformed code before exec.
Tests: semicolon repro above, indented import inside यदि, from os import path variants — all in tests/test_security.py (per CONTRIBUTING.md this touches the sandbox, so a Security Considerations section is required in the PR).
- 主要言語
- Python
- スター
- 1
- フォーク
- 4
- 平均マージ
- 12日 21時間
- マージ済み PR（30日）
- 1
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
alphacrack/python-maithili-dsl のほかの issue
-
help wanted infra P2 testing
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
-
documentation enhancement good first issue help wanted P3
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
documentation good first issue P3
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
-
area:linter enhancement good first issue P3 translation
難易度 2/5 半日 初心者へのやさしさ 66/100
-
area:cli enhancement good first issue P3
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
alphacrack/python-maithili-dsl の issue をすべて見る
似ている issue
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
JheysonJhordyHuarauyaRiquelme/sprocguard#18 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
dlq/synthpopcan#59 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
mdn/translated-content#40220 ·
メンテナーはふだん 4 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
ETSInitiative/PETSIRD#243 ·
メンテナーはふだん 1 日以内に返信
-
deferred p2
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
cuadrao-ai/argus#2160 ·
メンテナーはふだん 1 日以内に返信