search

ARUP-CAS / aiscr-webamcr

Archaeological Map of the Czech Republic (AMCR)
https://amcr-info.aiscr.cz/
GNU General Public License v3.0
5 stars 0 forks source link

Prověřit ERROR z logování TR #2215

Closed motyc closed 4 months ago

motyc commented 4 months ago

Vybráno typově, některé se opakují:

jhavrlant commented 4 months ago

@pesikj Nebylo by vhodné smazat logování, které obsahuje stack např:
stack = inspect.stack() caller = [x for x in stack] logger.debug("core_repository_connector.init.end", extra={"caller": caller, "transaction": self.transaction_uid, "ident_cely": record.ident_cely}) Zdá se mi, že toho logujeme zbytečně moc.

jhavrlant commented 4 months ago
  1. a 3. error pravděpodobně souvisí s https://github.com/dropzone/dropzone/issues/1956 generování miniatury z tiffu.
jhavrlant commented 4 months ago
  1. opravena #2216
jhavrlant commented 4 months ago

@motyc 5. chyba je způsobená zadáním datumu v oznamovacím formuláři jen 7.8.2024 bez rozmezí. Má se vrátit chyba aby to uživatel opravil nebo bude interval 1 den?

motyc commented 4 months ago

@jhavrlant Dal bych interval 1 den.

jhavrlant commented 4 months ago

@motyc Předposlední 2 chyby jsou pravděpodobně způsobeny http://192.168.254.34:5601/app/discover#/doc/858d02c4-091e-44c1-9600-ddb1a3dbaf32/amcr?id=6OVI_pABbIKW8Eg7hB0M , neuzavřená transakce fedory. Jenom jsem se chtěl zeptat proč informace o projektu C-202406009 se posílají na adresu archeolog@arup.cas.cz, nenašel jsem tam žádnou vazbu.

jhavrlant commented 4 months ago

Poslední chyba je stejná jako první.

motyc commented 4 months ago

@jhavrlant Protože daný účet má nastaveny notifikace na nové projekty (ve správě uživatelského účtu: https://amcr-test.aiscr.cz/notifikace-projekty/).

Ty neuzavřené transakce - víme, při čem vznikly? To by chtělo vystopovat. V Aplikaci nevisí a nic sem na TR teď myslím nezavíral.

motyc commented 4 months ago

Po dohodě s @jhavrlant je třeba ještě dodělat odchycení chyby pro mailer.

jhavrlant commented 4 months ago

@motyc Tak ono je ten mailer ve skutečnosti zachycený, ale zmátla mě ta hláška error. Přehodil bych to na warning.

motyc commented 4 months ago

@jhavrlant Ok, souhlas.