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

Нечувствительный к регистру эндпоинт юзеров #1035

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kix
Copy link

@kix kix commented Jan 5, 2023

УРЛы профилей будут открываться независимо от регистра: https://vas3k.club/user/vas3k/ откроет тот же профиль, что и https://vas3k.club/user/Vas3k/

slugify регистронезависим, так что вроде ничего не должно сломаться.

Fixes #886

@Programistich
Copy link
Collaborator

#886 залинкую

@dolshevsk
Copy link
Contributor

Не уверен, что это ничего не сломает. В самой моделе нет валидации и доп логики на регистронезависимость, и есть сценарии, где можно передать любой слаг, а модель его таким в базу и положит.

Даже у меня в профиле слаг имеет ловер и аперкейс, и юзер спокойно может создать такой же ник но с ловеркейсами, что в целом не проблема, но после твоего фикса это сломается.

@kix
Copy link
Author

kix commented Jan 5, 2023

@dolshevsk, да, тогда, возможно, стоит глянуть, что там с уникальностью ников с учётом регистронезависимости, и может докинуть ещё миграцию и фикс, чтобы быть уверенными в уникальности

@dolshevsk
Copy link
Contributor

Можно, но зачем? Какую проблему мы решаем?) Что плохого в том, что регистр учитывается?

@kix
Copy link
Author

kix commented Jan 5, 2023

@dolshevsk, я просто взял тикет и фигачу, чтобы немножко мозг развеять и залезть обратно в Джангу после нескольких лет :)
Если тикет нерелевантен, можно закрыть.

@Programistich
Copy link
Collaborator

У нас основная проблема не в том, что не кому закрывать тикеты, а в том, насколько нужны/не нужны/что с дизайном/етс

@skywinder
Copy link
Collaborator

@kix, привет. Я решил пройтись по старым задачам. Исправление может быть полезным, но, как заметили, важно проверять это и при создании профиля, чтобы не было конфликтов имен. Я не знаю, актуально ли это сейчас, можно попробовать доделать и проверить, или просто забить на это как на минорный баг. Выбирай.

@skywinder
Copy link
Collaborator

С одной стороны, я согласен, удобнее без учёта регистра (я сам при логине свой ник когда-то указал с большой буквы и теперь каждый раз путаюсь, когда ввожу его с маленькой и система не находит пользователя).
С другой стороны, это потенциально создаёт конфликты имен. Но я думаю, что правильно было бы сделать так, чтобы нельзя было создать двух пользователей с никами "aaa" и "AAA". Так что в общем и целом, это кажется правильным исправлением, если добавить проверки на этапе создания пользователя.

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.

Баг: регистрозависимый эндпоинт /user
4 participants