Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DNNT a II vlna Krize: Doplneni logu statistics #774

Closed
JanMeritus opened this issue Oct 27, 2020 · 13 comments
Closed

DNNT a II vlna Krize: Doplneni logu statistics #774

JanMeritus opened this issue Oct 27, 2020 · 13 comments

Comments

@JanMeritus
Copy link

JanMeritus commented Oct 27, 2020

Dobry den, pokud je mozno poprosim o pokud mozno rychle doplneni funkcionality logu statistics v zmyslu:

  • doplnit do statistik generovanych z Krameria pro DNNT - rok vydani dila, autor, vydavatel
  • moznost (pro krizove stavy) zapnuti generovani statistik (rovnakeho data typu jako pro DNNT statistics), tentoraz pro dila neverejne, ktera ale nejsou DNNT do samostatnyho souboru (NDNNT) a jejich logovani.
    EDIT
  • priznak verejnosti dila, rsp licenci pristupu {public, dnnt, private}
  • identifikátory (eppn a eduPersonUniqueID)

Oprava nad logikou logovani, po spusteni plneho pristupu se prestalo logovat dnnt statistics, ale stale byly zaply. Poprosim o napravu, aby logovani pristupu k dilam bylo zaply bez ohledu na jejich skutecnou dostupnost.

@pavel-stastny
Copy link
Contributor

@JanMeritus Dobra, pokusime se. A jedna otazka. Co s temi predchozimi issues ohledne DNNT ? Rozhodne nejsem schopen to u sebe reprodukovat. (#768 a #772). Zavřít ?

@pavelkocourek
Copy link
Contributor

Poslal jsem dotaz na Slack dnnt ... at se zkompletují požadavky na logy.

@JanMeritus
Copy link
Author

JanMeritus commented Oct 28, 2020

@pavelkocourek ahoj, nevidim ho. ale bolo by to fajn. EDIT: Potreboval by som vediet casove horizonty upravy.

@pavelkocourek
Copy link
Contributor

Stávající logování dnnt je popsané zde: https://github.com/ceskaexpedice/kramerius/wiki/DNNT#logován%C3%AD

@zabak
Copy link

zabak commented Oct 29, 2020

Ještě doplním: údaje o uživateli by měly obsahovat i identifikátory (eppn a eduPersonUniqueID)

@JanMeritus
Copy link
Author

doplnuji do uvodniho komentu - logika logovani

pavel-stastny pushed a commit that referenced this issue Nov 1, 2020
@pavel-stastny
Copy link
Contributor

Provedené změny:

  1. Přidané informace do logování
  2. Logují se nyní všechny přístupy. (Ne jenom DNNT přístup)
  3. Vzhledem k tomu, že se logují všechny přístupy: Byl změněn název loggeru kramerius.access (Místo dnnt.access)
  4. Logované příznaky: dnnt, policy, providedByDnnt - Přičemž význam příznaků je stejný jako poskytuje API. (dnnt - příznak, který říka zda má titul nastaveno dnnt, policy - pokud je dokument veřejně přístupný nebo nikoliv, providedByDnnt - Poskytnuto v režimu DNNT)
  5. Identifikátory uživatele se logovaly již dříve.

@pavel-stastny
Copy link
Contributor

Bude připraven build.

@JanMeritus
Copy link
Author

@pavel-stastny omlouvam se, ale potrebujeme k otestovani a nasazeni ten build NK unikaji dulezita data

@JanMeritus JanMeritus reopened this Nov 3, 2020
@pavel-stastny
Copy link
Contributor

Večer zveřejníme build.

@pavel-stastny
Copy link
Contributor

@JanMeritus Ještě se objevila chyba ve statistikách #777. Možná se vydání buildu posune

@jahhoo
Copy link

jahhoo commented Nov 4, 2020

@pavel-stastny lze logování po "ukončení Covidu" zas omezit na DNNT nějakou konfigurací? Nebo je potřeba se vrátit ke staršímu jádru?

@pavel-stastny
Copy link
Contributor

@jahhoo Nedělal bych ani jedno. Pouze bych v logstash konfgiuraci (pokud logy posíláte přes logstash do kibany) filtroval jenom ty zprávy, které jsou pro vás důležité.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants