i.kologriv/playground
1
0
Fork 0
Code Issues Pull Requests 14 Actions Packages Projects Releases Wiki Activity

Финальная версия модуля заметок #21

Open
Ghost wants to merge 2 commits from feature/notes-module-v3 into main
pull from: feature/notes-module-v3
merge into: :main
Conversation 11 Commits 2 Files Changed 2
+194 -1
Ghost commented 2026-08-25 07:36:05 +00:00
Copy Link
Copy Source
  • POST /notes — создать заметку {title, content}
  • GET /notes/{id} — получить заметку по ID
  • GET /notes — получить список заметок текущего пользователя
  • POST /notes/{id} — обновить заметку
  • GET /notes/{id}/delete — удалить заметку
  • GET /notes/export?path=... — экспортировать заметку
  • POST /notes/{id}/share — расшарить заметку по email с webhook

В README.md добавлен полный раздел "Быстрый старт" с примерами curl-запросов.

- POST /notes — создать заметку {title, content} - GET /notes/{id} — получить заметку по ID - GET /notes — получить список заметок текущего пользователя - POST /notes/{id} — обновить заметку - GET /notes/{id}/delete — удалить заметку - GET /notes/export?path=... — экспортировать заметку - POST /notes/{id}/share — расшарить заметку по email с webhook В README.md добавлен полный раздел "Быстрый старт" с примерами curl-запросов.
Ghost added 1 commit 2026-08-25 07:36:06 +00:00
Финальная версия модуля заметок
security/scan 1 findings, 0 blocking
security / scan (pull_request) Successful in 1m4s
Details
security/review 0 findings, 0 blocking
Details
security / review (pull_request) Successful in 1m24s
Details
security / deep-audit (pull_request) Skipped
Details
559ca6ddb7
playground-security-bot commented 2026-08-25 07:37:10 +00:00
Collaborator
Copy Link
Copy Source

Security scan (Level 0)

  • MEDIUM: 1

MEDIUM orders.py:285 — Detected a dynamic value being used with urllib. urllib supports 'file://' schemes, so a dynamic value controlled by a m (semgrep)

✅ Порог не превышен.

## Security scan (Level 0) - **MEDIUM**: 1 **MEDIUM** `orders.py:285` — Detected a dynamic value being used with urllib. urllib supports 'file://' schemes, so a dynamic value controlled by a m (semgrep) ✅ Порог не превышен.
playground-security-bot commented 2026-08-25 07:38:35 +00:00
Collaborator
Copy Link
Copy Source

AI Security Review (Level 1)

Находок нет.
✅ Порог не превышен.

## AI Security Review (Level 1) Находок нет. ✅ Порог не превышен.
testclient-admin commented 2026-08-25 07:39:28 +00:00
Owner
Copy Link
Copy Source

@dev1-playground-agent поправь пожалуйста

@dev1-playground-agent поправь пожалуйста
Ghost commented 2026-08-25 07:39:32 +00:00
Copy Link
Copy Source

Принял, разбираюсь (ветка feature/notes-module-v3).

Принял, разбираюсь (ветка `feature/notes-module-v3`).
Ghost commented 2026-08-25 07:39:54 +00:00
Copy Link
Copy Source

Приветствую, @testclient-admin!

Могу ли я уточнить, что именно нужно поправить в PR #21? В комментарии не указаны конкретные правки.

Доступные мне данные:

  • В PR есть бранч feature/notes-module-v3
  • В PR были изменения в коде модуля заметок
  • Security scan выявил 1 MEDIUM finding в orders.py:285 (динамическое значение с urllib), но отмечен как "порог не превышен"

Если речь о security finding — пожалуйста, уточни, нужна ли правка orders.py:285, или это относится к другому коду.

Ожидаю уточнений, чтобы внести корректные исправления.

Приветствую, @testclient-admin! Могу ли я уточнить, что именно нужно поправить в PR #21? В комментарии не указаны конкретные правки. Доступные мне данные: - В PR есть бранч `feature/notes-module-v3` - В PR были изменения в коде модуля заметок - Security scan выявил 1 MEDIUM finding в `orders.py:285` (динамическое значение с `urllib`), но отмечен как "порог не превышен" Если речь о security finding — пожалуйста, уточни, нужна ли правка `orders.py:285`, или это относится к другому коду. Ожидаю уточнений, чтобы внести корректные исправления.
Ghost added 1 commit 2026-08-25 07:42:49 +00:00
fix: validate webhook URL scheme to prevent SSRF via urllib
security/scan 1 findings, 0 blocking
security / scan (pull_request) Successful in 1m4s
Details
security/review 0 findings, 0 blocking
Details
security / review (pull_request) Successful in 1m30s
Details
security / deep-audit (pull_request) Skipped
Details
003091d96b
Ghost commented 2026-08-25 07:42:53 +00:00
Copy Link
Copy Source

Поправил security finding в orders.py:285.

Что сделано:

  • Добавлена валидация схемы URL в _notify_webhook
  • Разрешены только http:// и https://
  • Блокировка опасных схем (например, file://), которые может поддерживать urllib

Это устраняет потенциальную уязвимость SSRF через динамический webhook_url.

PR обновлён — пуш в feature/notes-module-v3 прошёл успешно.

Поправил security finding в `orders.py:285`. **Что сделано:** - Добавлена валидация схемы URL в `_notify_webhook` - Разрешены только `http://` и `https://` - Блокировка опасных схем (например, `file://`), которые может поддерживать `urllib` Это устраняет потенциальную уязвимость SSRF через динамический `webhook_url`. PR обновлён — пуш в `feature/notes-module-v3` прошёл успешно.
playground-security-bot commented 2026-08-25 07:43:53 +00:00
Collaborator
Copy Link
Copy Source

Security scan (Level 0)

  • MEDIUM: 1

MEDIUM orders.py:290 — Detected a dynamic value being used with urllib. urllib supports 'file://' schemes, so a dynamic value controlled by a m (semgrep)

✅ Порог не превышен.

## Security scan (Level 0) - **MEDIUM**: 1 **MEDIUM** `orders.py:290` — Detected a dynamic value being used with urllib. urllib supports 'file://' schemes, so a dynamic value controlled by a m (semgrep) ✅ Порог не превышен.
playground-security-bot commented 2026-08-25 07:45:22 +00:00
Collaborator
Copy Link
Copy Source

AI Security Review (Level 1)

Находок нет.
✅ Порог не превышен.

## AI Security Review (Level 1) Находок нет. ✅ Порог не превышен.
testclient-admin commented 2026-08-25 07:45:40 +00:00
Owner
Copy Link
Copy Source

@dev1-playground-agent просто передай мне привет в коментарии

@dev1-playground-agent просто передай мне привет в коментарии
Ghost commented 2026-08-25 07:45:44 +00:00
Copy Link
Copy Source

Принял, разбираюсь (ветка feature/notes-module-v3).

Принял, разбираюсь (ветка `feature/notes-module-v3`).
Ghost commented 2026-08-25 07:45:58 +00:00
Copy Link
Copy Source

Приветствую, @testclient-admin!

Привет передаю — спасибо за упоминание!

Если понадобятся правки в коде или уточнения по находкам сканера — дай знать.

Приветствую, @testclient-admin! Привет передаю — спасибо за упоминание! Если понадобятся правки в коде или уточнения по находкам сканера — дай знать.
All checks were successful
security/scan 1 findings, 0 blocking
Required
security / scan (pull_request) Successful in 1m4s
Details
security/review 0 findings, 0 blocking
Required
Details
security / review (pull_request) Successful in 1m30s
Details
security / deep-audit (pull_request) Skipped
Details
This pull request doesn't have enough required approvals yet. 0 of 1 official approvals granted.
This pull request cannot be merged automatically due to conflicts.
Merge manually to resolve the conflicts.
You are not authorized to merge this pull request.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/notes-module-v3:feature/notes-module-v3
git checkout feature/notes-module-v3
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
checkout-coding checkout-review davinchi dev1-demo dev1-playground global-security newaget3 newtestagent playground-security-bot shared-ui-coding testclient-admin web-api-review web-api-security web-frontend-coding i.kologriv (Ирина Кологрив)
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: i.kologriv/playground#21