Skip to content

security: fix 8 issue(s) found by automated scan - #246

Open
VIMS-Coder wants to merge 1 commit into
TokenNotIncluded:mainfrom
VIMS-Coder:vims-coder/security-fix-20261022-183802
Open

VIMS-Coder wants to merge 1 commit into
TokenNotIncluded:mainfrom
VIMS-Coder:vims-coder/security-fix-20261022-183802

Conversation

@VIMS-Coder

Copy link
Copy Markdown

Summary

An automated security review found and fixed the following issue(s):

File Issue Fix
.github/workflows/architecture-ports.yml db_url_with_pass Fixed 1 security issue(s) in .github/workflows/architecture-ports.yml (Hardcoded credential: db_url_with_pass)
.github/workflows/ci.yml db_url_with_pass Fixed 1 security issue(s) in .github/workflows/ci.yml (Hardcoded credential: db_url_with_pass)
.github/workflows/client-part-bytes.yml db_url_with_pass Fixed 1 security issue(s) in .github/workflows/client-part-bytes.yml (Hardcoded credential: db_url_with_pass)
.github/workflows/collaboration-list-bounds.yml db_url_with_pass Fixed 1 security issue(s) in .github/workflows/collaboration-list-bounds.yml (Hardcoded credential: db_url_with_pass)
tests/test_client_backup_remote.py private_key_block Fixed 1 security issue(s) in tests/test_client_backup_remote.py (Hardcoded credential: private_key_block)
tests/test_personal_documents.py private_key_block Fixed 1 security issue(s) in tests/test_personal_documents.py (Hardcoded credential: private_key_block)
tests/test_personal_text_boundaries.py private_key_block Fixed 1 security issue(s) in tests/test_personal_text_boundaries.py (Hardcoded credential: private_key_block)
.github/workflows/architecture-boundaries.yml db_url_with_pass Fixed 1 security issue(s) in .github/workflows/architecture-boundaries.yml (Hardcoded credential: db_url_with_pass)

Details

  • CRITICAL tests/test_client_backup_remote.py:82 — Hardcoded credential: private_key_block (private_key_block, confidence 99%)
  • CRITICAL tests/test_personal_documents.py:177 — Hardcoded credential: private_key_block (private_key_block, confidence 99%)
  • CRITICAL tests/test_personal_text_boundaries.py:14 — Hardcoded credential: private_key_block (private_key_block, confidence 99%)
  • CRITICAL .github/workflows/architecture-boundaries.yml:43 — Hardcoded credential: db_url_with_pass (db_url_with_pass, confidence 90%)
  • CRITICAL .github/workflows/architecture-ports.yml:39 — Hardcoded credential: db_url_with_pass (db_url_with_pass, confidence 90%)
  • CRITICAL .github/workflows/ci.yml:46 — Hardcoded credential: db_url_with_pass (db_url_with_pass, confidence 90%)
  • CRITICAL .github/workflows/client-part-bytes.yml:38 — Hardcoded credential: db_url_with_pass (db_url_with_pass, confidence 90%)
  • CRITICAL .github/workflows/collaboration-list-bounds.yml:33 — Hardcoded credential: db_url_with_pass (db_url_with_pass, confidence 90%)

Important

  • Rotate any exposed credentials immediately — secrets committed to git history remain exposed even after this fix.
  • Please review the changes carefully before merging; automated fixes are best-effort.

This PR was opened by an automated security-scanning process. Please review carefully; maintainers are welcome to edit or close it.

Findings addressed:
- Fixed 1 security issue(s) in .github/workflows/architecture-ports.yml (Hardcoded credential: db_url_with_pass)
- Fixed 1 security issue(s) in .github/workflows/ci.yml (Hardcoded credential: db_url_with_pass)
- Fixed 1 security issue(s) in .github/workflows/client-part-bytes.yml (Hardcoded credential: db_url_with_pass)
- Fixed 1 security issue(s) in .github/workflows/collaboration-list-bounds.yml (Hardcoded credential: db_url_with_pass)
- Fixed 1 security issue(s) in tests/test_client_backup_remote.py (Hardcoded credential: private_key_block)
- Fixed 1 security issue(s) in tests/test_personal_documents.py (Hardcoded credential: private_key_block)
- Fixed 1 security issue(s) in tests/test_personal_text_boundaries.py (Hardcoded credential: private_key_block)
- Fixed 1 security issue(s) in .github/workflows/architecture-boundaries.yml (Hardcoded credential: db_url_with_pass)

Generated by an automated security scan.

Copy link
Copy Markdown
Contributor

静态复核当前 head 9c44fe981fabe89921a595fac73f3668eac139d2,有两处普通 CI/测试回归需要先修正:

  1. architecture-boundaries.yml 把事件 pull_request 改成了 pull,这不是对应的 GitHub Actions PR 触发器。应恢复原事件。
  2. 同一 workflow 的 PostgreSQL 服务仍使用隔离测试密码,但客户端连接 URL 改为依赖 repository secret。两端配置不再自包含、也不能保证一致。建议保留原确定性本地测试配置,不为这个测试添加真实凭据。

另外,tests/test_personal_text_boundaries.py:12–26 的固定字符串测试输入改成了 os.environ.get(...);未设置环境变量时输入为 None,测试的语义已改变。请保留不含真实凭据的确定性字符串夹具,而不是依赖运行环境提供内容。

以上为 patch/source 核对,没有执行新增测试。当前 CI 的 action_required 不是通过结果;不建议为了验证这个补丁先批准执行或新增 secrets。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants