docs(env): udokumentuj DJANGO_BPP_ROLLBAR_IGNORE_SMTP_AUTH_ERRORS#22
Open
mpasternak wants to merge 1 commit into
Open
docs(env): udokumentuj DJANGO_BPP_ROLLBAR_IGNORE_SMTP_AUTH_ERRORS#22mpasternak wants to merge 1 commit into
mpasternak wants to merge 1 commit into
Conversation
Nowa zmienna w BPP (iplweb/bpp#678) wycisza w Rollbarze smtplib.SMTPAuthenticationError na konkretnej instalacji. Powod: na bpp.umlub.pl serwer SMTP uczelni odrzuca poswiadczenia (535 5.7.3), co jest awaria po stronie administratorow poczty klienta - nie naprawimy tego kodem, a kazda nieudana wysylka powiadomienia zasila ten sam item w Rollbarze. Zmiana jest wylacznie dokumentacyjna. Zmienna nalezy do konfiguracji aplikacji, czyli ${BPP_CONFIGS_DIR}/.env, a nie do tego repo - wpisanie jej tutaj wlaczyloby wyciszenie WSZEDZIE. Wzorzec komentarza w .env.sample opisujacego zmienna z drugiego pliku jest juz w repo uzywany dla DJANGO_BPP_HTML2DOCX_URL. Nie trzeba ruszac compose'ow: appserver (docker-compose.application.yml) i workerserver (docker-compose.workers.yml) ladują ${BPP_CONFIGS_DIR}/.env przez `env_file`, wiec zmienna dociera do obu automatycznie. Worker jest tu istotny - wyjatek leci z djcelery_email, nie z requestu. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NcAqeqyqBzNEkkVnhpHDaH
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Co i po co
BPP dostaje nową zmienną
DJANGO_BPP_ROLLBAR_IGNORE_SMTP_AUTH_ERRORS(iplweb/bpp#678), która wycisza w Rollbarze
smtplib.SMTPAuthenticationErrorna konkretnej instalacji.
Powód: na
bpp.umlub.plserwer SMTP uczelni odrzuca poświadczenia(
535 5.7.3 Authentication unsuccessful). To awaria po stronie administratorówpoczty klienta — nie naprawimy jej kodem, a każda nieudana wysyłka powiadomienia
zasila ten sam item w Rollbarze (już 59 wystąpień).
Zmiana jest wyłącznie dokumentacyjna
Zmienna należy do konfiguracji aplikacji, czyli
${BPP_CONFIGS_DIR}/.env,a nie do tego repo — wpisanie jej tutaj włączyłoby wyciszenie wszędzie.
Dodaję więc tylko blok komentarza w
.env.sample, zgodnie ze wzorcem, któryrepo już stosuje dla
DJANGO_BPP_HTML2DOCX_URL(też zmienna z drugiego pliku,opisana tutaj dla odkrywalności).
Compose bez zmian — sprawdzone
appserver(docker-compose.application.yml:89) iworkerserver(
docker-compose.workers.yml:27) ładują${BPP_CONFIGS_DIR}/.envprzezenv_file, więc zmienna dociera do obu automatycznie.Worker jest tu istotny: wyjątek leci z
djcelery_emailw kolejce Celery,a nie z requestu — ustawienie samego appservera nic by nie dało.
Krok wdrożeniowy (do wykonania ręcznie na serwerze UM Lublin)
…po czym restart
appserveriworkerserver. Ten PR sam z siebie niczego niewłącza.
Zastrzeżenie
To nie naprawia poczty. Dopóki klient nie odblokuje SMTP-a, powiadomienia
o zgłoszeniach publikacji nie docierają do redakcji (pięć adresów
@umlub.edu.pl),a zgłoszenia leżą w bazie niezauważone. Po naprawie po stronie klienta zmienną
należy usunąć, żeby odzyskać sygnał o awariach poczty — jest o tym mowa
w komentarzu.
🤖 Generated with Claude Code
https://claude.ai/code/session_01NcAqeqyqBzNEkkVnhpHDaH