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

feat: add basic auth #67

Merged
merged 1 commit into from Apr 23, 2022
Merged

feat: add basic auth #67

merged 1 commit into from Apr 23, 2022

Conversation

Raerten
Copy link
Contributor

@Raerten Raerten commented Apr 22, 2022

Добавлена настройка для включения basic авторизации

в настройка добавлено поле для ввода закодированных логина и пароля

image

как получить закодированные логина и пароля:
пишем логин и пароль через двоеточие login:password и эту строку кодируем в base64

image

после этого на всех страницах будет предложение ввести пароль

image

возможно стоит вывести его в лог при загрузке (не сделано).

так же просьба проверить правильность длин буферов для пароля и поля в nvs (на C не пишу, мог ошибиться)

@Raerten
Copy link
Contributor Author

Raerten commented Apr 22, 2022

basic авторизация защищает от попадания на страницу управления случайных людей, но не защищает от перехвата трафика,
но, как уже писал alutov, мы считаем что работаем подконтрольном контуре и защита от взлома не нужна, digest авторизация сильно усложнит код

@alutov
Copy link
Owner

alutov commented Apr 23, 2022

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

@alutov alutov merged commit 26af195 into alutov:master Apr 23, 2022
@Raerten
Copy link
Contributor Author

Raerten commented Apr 23, 2022

Просмотрел, вроде ничего такого не нашел. NVS при записи ничего не требует, а при чтении задается максимальный размер буфера, если не лезет, то не читается вообще ничего. Я так понимаю, этот код уже работает?

код у меня работает второй день так что он не супер провереный. и на С я не пишу, поэтому мог ошибиться где-то в розмерах буферов

И еще, я так понимаю, надо завести ветку dev и туда грузить локальный код, а потом сливать изменения в основную ветку и этого достаточно.

можно через дев, если исправлений по pull реквесту не было, можно сразу в местер.

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.

None yet

2 participants