mam/mamweb
11
0
Fork 0
Code Issues Pull requests 8 Projects Wiki Activity Actions

Podezřelé semináře (#1465) #65

Merged
zelvuska merged 27 commits from podezrele-seminare into master 2024-11-05 20:31:55 +01:00
Conversation 9 Commits 27 Files changed 102 +9 -9
3 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit eb6eb2d6fb - Show all commits

4
mamweb/settings_common.py
View file

@ -279,11 +279,11 @@ LOGGING = {
'filters': ['Http404AsInfo'],
},
'seminar.prihlaska.form':{
'personalni.prihlaska.form':{
'handlers': ['console','registration_logfile'],
'level': 'INFO'
},
'seminar.prihlaska.problem':{
'personalni.prihlaska.problem':{
'handlers': ['console','mail_registration','registration_error_log'],
'level': 'INFO'
},

4
personalni/forms.py
View file

@ -27,7 +27,7 @@ class TelInput(forms.TextInput):
class UdajeForm(forms.Form):
username = None
err_logger = logging.getLogger('seminar.prihlaska.problem')
err_logger = logging.getLogger('personalni.prihlaska.problem')
jmeno = forms.CharField(label='Jméno', max_length=256, required=True)
prezdivka_resitele = forms.CharField(label='Přezdívka (veřejná)', max_length=256, required=False)
@ -147,7 +147,7 @@ class PrihlaskaForm(PasswordResetForm, UdajeForm):
class ProfileEditForm(UdajeForm):
err_logger = logging.getLogger('seminar.edit.problem')
err_logger = logging.getLogger('personalni.edit.problem')
ledoian commented 2024-11-03 01:31:40 +01:00
Outdated
Review
Copy link

Heh, tohle nikde není deklarované, i když se to od pohledu má chovat (zvlášť co se týče ochrany dat) stejně jako personalni.prihlaska.problem. Myslím, že by mělo stačit tady (jinde to stejně není) přejmenovat na personalni.prihlaska.problem.edit (ale teď nekontroluji, jestli si pamatuji chování pythoního logování správně…)

Heh, tohle nikde není deklarované, i když se to od pohledu má chovat (zvlášť co se týče ochrany dat) stejně jako `personalni.prihlaska.problem`. Myslím, že by mělo stačit tady (jinde to stejně není) přejmenovat na `personalni.prihlaska.problem.edit` (ale teď nekontroluji, jestli si pamatuji chování pythoního logování správně…)
zelvuska commented 2024-11-05 15:16:48 +01:00
Outdated
Review
Copy link

To nebylo ani předtím… Myslíš, že personalni.prihlaska.problem.edit pomůže?

To nebylo ani předtím… Myslíš, `že personalni.prihlaska.problem.edit` pomůže?
ledoian commented 2024-11-05 16:00:20 +01:00
Outdated
Review
Copy link

Mám za to, že Pythoní logování drží hierarchii podle teček, takže personalni.prihlaska.problem.edit se chová by default stejně jako personalni.prihlaska.problem. Ale nevěřím tomu úplně, tak to asi na webschůzce vyzkouším na testwebu (ze shellu)…

Mám za to, že Pythoní logování drží hierarchii podle teček, takže `personalni.prihlaska.problem.edit` se chová by default stejně jako `personalni.prihlaska.problem`. Ale nevěřím tomu úplně, tak to asi na webschůzce vyzkouším na testwebu (ze shellu)…
username = forms.CharField(
label='Přihlašovací jméno',
max_length=256,

10
personalni/views.py
View file

@ -161,7 +161,7 @@ def prihlaska_log_gdpr_safe(logger, gdpr_logger, msg, form_data):
@sensitive_post_parameters('jmeno', 'prijmeni', 'email', 'telefon', 'datum_narozeni', 'ulice', 'mesto', 'psc', 'skola')
def resitelEditView(request):
err_logger = logging.getLogger('seminar.prihlaska.problem')
err_logger = logging.getLogger('personalni.prihlaska.problem')
## Načtení objektů Osoba a Resitel patřících k aktuálně přihlášenému uživateli
u = request.user
osoba_edit = m.Osoba.objects.get(user=u)
@ -199,7 +199,7 @@ def resitelEditView(request):
## Změny v osobě
fcd = form.cleaned_data
form_hash = hash(frozenset(fcd.items()))
form_logger = logging.getLogger('seminar.prihlaska.form')
form_logger = logging.getLogger('personalni.prihlaska.form')
form_logger.info("EDIT:" + str(fcd) + str(form_hash)) # TODO možná logovat jinak
osoba_edit.jmeno = fcd['jmeno']
osoba_edit.prijmeni = fcd['prijmeni']
@ -248,9 +248,9 @@ def resitelEditView(request):
@sensitive_post_parameters('jmeno', 'prijmeni', 'email', 'telefon', 'datum_narozeni', 'ulice', 'mesto', 'psc', 'skola', 'jak_se_dozvedeli')
def prihlaskaView(request):
generic_logger = logging.getLogger('seminar.prihlaska')
err_logger = logging.getLogger('seminar.prihlaska.problem')
form_logger = logging.getLogger('seminar.prihlaska.form')
generic_logger = logging.getLogger('personalni.prihlaska')
err_logger = logging.getLogger('personalni.prihlaska.problem')
form_logger = logging.getLogger('personalni.prihlaska.form')
if request.method == 'POST':
form = PrihlaskaForm(request.POST)
# TODO vyresit, co se bude v jakych situacich zobrazovat