Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Проверка чтобы не было запущено несколько терминалов с одним токеном #57

Closed
mrerro opened this issue May 31, 2023 · 0 comments · Fixed by #76
Assignees
Labels
good first issue 👶 Задача подходит для новичков new feature 🆕 Новая фича или запрос на нее

Comments

@mrerro
Copy link
Member

mrerro commented May 31, 2023

Добавить в вебсокет токен терминала на;

  1. валидность
  2. что терминал с таким токеном уже запущен
    Если токена нет у нас в базе или такой терминал уже запущен, выслать ответ на закрытие этого терминала.

print-winapp profcomff/print-winapp#55 (comment)

@mrerro mrerro added new feature 🆕 Новая фича или запрос на нее good first issue 👶 Задача подходит для новичков labels May 31, 2023
@dyakovri dyakovri moved this to Backlog in Viribus Team Jun 2, 2023
@dyakovri dyakovri mentioned this issue Dec 24, 2023
3 tasks
@dyakovri dyakovri self-assigned this Dec 24, 2023
@dyakovri dyakovri moved this from Backlog to In Progress in Viribus Team Dec 24, 2023
dyakovri added a commit that referenced this issue Dec 24, 2023
## Изменения
Resolves #57

## Детали реализации
- При подключении по WebSocket необходимо передать Header
"Authorization" с значением токена AuthAPI
- Если токен неверный – будет выдана ошибка `Not authenticated`
- Если у токена недостаточно прав – будет выдана ошибка `Unauthorized`
- Если токен уже использовался в течение последней минуты – будет выдана
ошибка `Already in use`
- Ошибка возвращается в теле первого сообщения от сокета в виде JSON
`{"error": "reason"}`, после чего сокет закрывается нормально
@github-project-automation github-project-automation bot moved this from In Progress to Done in Viribus Team Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue 👶 Задача подходит для новичков new feature 🆕 Новая фича или запрос на нее
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants