Conversation
maxjamchuk
left a comment
maxjamchuk
left a comment
There was a problem hiding this comment.
Главное разобраться с тем что в итоге возвращаем в AddChannelView.get() и почистить .env.example
|
|
||
| TELEGRAM_API_ID=1234567890 | ||
| TELEGRAM_SESSION=session.session | ||
| TELEGRAM_API_HASH=hfk693kcln9673nclbb94j |
There was a problem hiding this comment.
Caution
даже если это значение не настоящее, а примерное, это провоцирует людей хранить/коммитить реальные секреты в .env.example, а потом случайно утекать ими в репозиторий/логи/скриншоты.
TELEGRAM_API_HASH — это по сути секрет (как API key).
Как лучше:
Оставить в .env.example только плейсхолдеры, явно пометить как секреты:
TELEGRAM_API_HASH=__CHANGE_ME__или что-то типа того.
если же это реальное значение, то его надо перевыпустить и больше не светить в GitHub.
| DATABASE_ENGINE="postgresql" | ||
|
|
||
| TELEGRAM_API_ID=1234567890 | ||
| TELEGRAM_SESSION=session.session |
There was a problem hiding this comment.
Tip
Название TELEGRAM_SESSION=session.session выглядит как имя файла” но переменная названа как session — значение.
Лучше: TELEGRAM_SESSION_FILE=session.session или TELEGRAM_SESSION_PATH=... — чтобы по названию было понятно, что это путь/файл, а не строка-сессия.
| postgres | ||
| key_word.py | ||
|
|
||
| session.session |
There was a problem hiding this comment.
Note
добавили session.session — СУПЕР, идея понятна: не коммитить файл сессии Telegram-клиента.
There was a problem hiding this comment.
Tip
в конце файла нет перевода строки.
в принципе ничего ужасного, прод не сломается, но будет постоянно шуметь в линтерах.
Фикс: добавить пустую строчку в конец файла.
| async def get(self, request, *args, **kwargs): | ||
| form = ChannelForm() | ||
| return JsonResponse({"status": "ok", "form_fields": list(form.fields.keys())}) | ||
| return JsonResponse(request, "Channels/Add", props={ |
There was a problem hiding this comment.
Caution
насколько я помню JsonResponse сам по себе не принимает (request, component, props)
здесь подробнее
фикс скорее всего будет примерно такой:
return render(request, "Channels/Add", props={...})3 participants
TELEGRAM_API_ID=1234567890
TELEGRAM_SESSION=session.session
TELEGRAM_API_HASH=hfk693kcln9673nclbb94j