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

Fix local jwt #191

Closed
wants to merge 3 commits into from
Closed

Fix local jwt #191

wants to merge 3 commits into from

Conversation

Takaiva
Copy link
Contributor

@Takaiva Takaiva commented Feb 3, 2023

Локально бэк не разворачивался из-за ошибки с JWT.
Приложение не могло корректно считать ключи из переменной окружения.

@fey
Copy link
Contributor

fey commented Feb 8, 2023

@Takaiva а в чем конкретно ошибка? По идее переменные должны подтягиваться из условного .env файла.

@Takaiva
Copy link
Contributor Author

Takaiva commented Feb 9, 2023

@fey По идее должны, но почему-то не подтягивались, не считывались с .env файла. Приложение падало с ошибкой секретного ключа и на все запросы ответ был 500. По сути, вот эти правки – это то, как я помог человеку продвинуться дальше "Ошибки 500" и вообще суметь зарегистрироваться в приложении, залогиниться и получиться доступ к редактору и профилю. Запустите локально, гляньте консоль "до" и "после" этих фиксов, там что-то странное, в чем я сам толком не разобрался.

@fey
Copy link
Contributor

fey commented Feb 9, 2023

Просто сейчас секрет статический и находится в коде, и получается что любой чувак может подделать jwt.
Такое может сработать для локального окружения но я так понимаю тут нет разделения и для прода и для дева все одинаково будет.

@dzencot
Copy link
Collaborator

dzencot commented Feb 14, 2023

Ребят, эту проблему нужно решать через переменные окружения, то есть нужно сделать так, чтобы переменные подцеплялись из .env файлов

@dzencot
Copy link
Collaborator

dzencot commented Feb 19, 2023

@Takaiva займешься этим? Нужно разобраться почему .env не подключается. Может подключить dotenv

@dzencot
Copy link
Collaborator

dzencot commented Feb 27, 2023

Я займусь

@dzencot dzencot closed this Mar 10, 2023
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

3 participants