Skip to content

Исправлены chat permissions для swagger schema#621

Merged
Toksi86 merged 1 commit intodevfrom
devops-structure-rework
Mar 27, 2026
Merged

Исправлены chat permissions для swagger schema#621
Toksi86 merged 1 commit intodevfrom
devops-structure-rework

Conversation

@Toksi86
Copy link
Copy Markdown
Collaborator

@Toksi86 Toksi86 commented Mar 27, 2026

Что изменено

  • Исправлена генерация Swagger schema после предыдущего PR:
    • staff-пользователь мог открыть /swagger/, но запрос /swagger/?format=openapi падал с 500
    • причина была в chats.permissions.IsChatMember, который ожидал kwargs["id"] даже во время schema generation
  • Переработаны chat permissions:
    • IsChatMember больше не падает на schema generation
    • IsProjectChatMember теперь корректно работает как с id, так и с pk
    • доступ к project chat теперь явно проверяется для leader и collaborator
  • Добавлены регрессионные тесты:
    • staff-доступ к /swagger/?format=openapi
    • доступ leader
    • доступ collaborator
    • запрет доступа для постороннего пользователя

@Toksi86 Toksi86 requested a review from TThanos3000 March 27, 2026 13:33
@Toksi86 Toksi86 merged commit 2c9e6de into dev Mar 27, 2026
2 checks passed
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