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.
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