search

hombit / sai-tasks2019

0 stars 18 forks source link

Запуск Марсианских новостей через wsgi #30

Closed hombit closed 4 years ago

hombit commented 4 years ago

В марсианских новостях #10 глобальная переменная с API-ключем используется внутри блока if __name__ == '__main__'. Во-первых, не стоит вводить глобальные переменные ниже кода, их использующего, еще и внутри условных операторов. Во-вторых при запуске через wsgi (а так должны в продакшене запускать все сервера на Flask) получается, что этой переменной не существует

Проще всего просто вынести код, который определяет NASA_API_key в начало файла, оставив его глобальным

Также стоит переименовать site.py во что-нибудь, что не перекликается со стандартной библиотекой

AndreyTavleev commented 4 years ago

Создал pull request с исправлением.

hombit commented 4 years ago

Fixed by #31