radar-parlamentar / radar

Estamos de mudança para o GitLab: https://gitlab.com/radar-parlamentar/radar.
http://radarparlamentar.polignu.org
84 stars 44 forks source link

Warning: DEBUG log level #440

Open leonardofl opened 6 years ago

leonardofl commented 6 years ago

Ao subir o celery (start_celery.sh), aparece o seguinte warning:

 WARNING/MainProcess] /home/leonardo/workspaces/radar/venv3/lib/python3.6/site-packages/celery/fixups/django.py:202: UserWarning: Using settings.DEBUG leads to a memory leak, never use this setting in production environments!
  warnings.warn('Using settings.DEBUG leads to a memory leak, never '

E de fato, debug não deve estar ligado em produção. Mesmo em desenvolvimento por padrão melhor deixar desligado.

diraol commented 6 years ago

Acho que este commit resolve o problema não? https://github.com/radar-parlamentar/radar/commit/1afeb1deaf7b9c73eeade347cb92407e3dcf263d

Apenas fiquei com uma dúvida. O problema é ter "settings.DEBUG = True" ou o problema é usar a constante "settings.DEBUG"?

diraol commented 6 years ago

@leonardofl agree que resolve?

leonardofl commented 6 years ago

De fato a msg não é clara, mas eu espero q o problema seja "settings.DEBUG = True". Nesse caso, em produção já sabemos q isso não será verdade, né? Então acho q fecha a issue.