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

Убрать ключ confirmPassword из схемы данных при запросах на бэкенд #352

Open
3 tasks
faciledictu opened this issue Aug 17, 2023 · 1 comment
Assignees
Labels

Comments

@faciledictu
Copy link
Contributor

Problem description

Remove confirmPassword from request schemas of some endpoints

Поправьте, если ошибаюсь.

Кажется, что отправлять на сервер пароль и его же подтверждение — это излишне. В любом случае, даже если где-то есть подтверждение пароля в форме, сличение двух паролей — это, наверное, задача фронтенда, а не сервера.

Плюс сейчас на фронте нет поля «подтвердить пароль». Схема подтверждения работает по принципу того, что пользователь может отобразить пароль с помощью кнопки.

Proposed solution

Касается двух эндпоинтов.

  1. POST /api/users (используется для регистрации пользователя)
    Изменить схему запроса:
    {
    "username": "JohnDoe",
    "email": "jane-doe@mail.ru",
    "password": "haew6wae56a45ewgd",
    "confirmPassword": «haew6wae56a45ewgd"
    }

  2. PUT /api/users/{id} (используется для обновления данных)
    Изменить схему запроса:
    {
    "username": "UpdatedJohnDoe",
    "email": "updated-jane-doe@mail.ru",
    "currPassword": "haew6wae56a45ewgd",
    "password": "ha6ew6ewa5gea",
    "confirmPassword": "ha6ew6ewa5gea"
    }

План

  • обновить схемы запросов на бэкенде, чтобы они не ожидали ключ confirmPassword и не сверяли его с password
  • обновить тесты на бэкенде
  • обновить код фронтенда, чтобы клиент не отсылал эти данные на сервер

Steps:

  1. Backend. Change request schemas according examples above and remove confirmPassword validation
  2. Backend. Update tests
  3. Frontend. Update requests on form submitting
@shahzod-davlatov
Copy link
Contributor

Возьму

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🏗 In progress
Development

No branches or pull requests

2 participants