Kaikki verkkokirjastossa tehdyt toimenpiteet eivät lokitu

KohaSuomi / Koha

(KOHA) Koha is a free software integrated library system (ILS). Koha is distributed under the GNU GPL version 3 or later. Note: this is a synced mirror of the official Koha repo.

https://koha-community.org/

GNU General Public License v3.0

7 stars 13 forks source link

Kaikki verkkokirjastossa tehdyt toimenpiteet eivät lokitu #473

Open sksandel opened 1 year ago

sksandel commented 1 year ago

Mikä vikana?

Suurin osa asiakkaan tekemistä muutoksista (Finnassa) ei näy Kohan lokissa

Ei näy lokissa:

kirjautuminen verkkokirjastoon
omien tietojen katselu (mikään välilehti)
varauksen teko
varauksen muokkaus
varauksen poisto
viestiasetusten muokkaus
tunnusluvun vaihto verkkokirjastossa
Osoite- tai varaustunnusmuutospyynnön lähettäminen (muutoksen tarkastelu ja hyväksyminen virkailijan toimesta näkyy)

Näkyy lokissa:

lainan uusinta
omien tietojen muokkaus: sähköposti ja puhelinnumero (loki näyttää myös vanhat ja uudet arvot)
lainaushistorian tallennusasetuksen muutos

Mitä pitäisi tapahtua

No response

Kuinka toistaa ongelma/asia

Testattu tuotannossa ja Nextillä.

Selain

No response

Jotain muuta?

No response

AnneliO commented 1 year ago

Varauksiin liittyvät toiminnot lokittuvat, jos päällä on HoldsLog. Se pitäisi olla kaikilla päällä. Näitä ei kuitenkaan saa Muutosloki-työkalulla helposti näkyville, koska varaukset menevät lokille varauksen id:n mukaan, ei asiakkaan borrowernumberin mukaan. Näitä voi hakea erillisellä SQL-kyselyllä, joka löytyy Valmiita SQL-raportteja -sivulta.

Toinen vaihtoehto on laittaa Työkaluissa Lokien katselu -työkalussa asiakkaan borrowernumber Tiedot-kenttään ja valita osioksi Varaukset.

sksandel commented 1 year ago

Ok hyvä huomio tuo varaustoimintojen näkyminen muualla. Päivitän tämän tiketin uusilla tiedoilla.

Ei näy lokissa:

kirjautuminen verkkokirjastoon
omien tietojen katselu (mikään välilehti)

Näkyy lokissa:

epäonnistunut kirjautumisyritys (väärä pin)
lainan uusinta
salasanan vaihto
viestiasetusten muokkaus
omien tietojen muokkaus: sähköposti ja puhelinnumero (loki näyttää myös vanhat ja uudet arvot)
lainaushistorian tallennusasetuksen muutos

Eli lähes kaikki kirjautuvat nyt lokiin. Mutta olisi hyvä jos verkkokirjastoon kirjautumiset lokittuisivat. Olisiko se mahdollista?

AnneliO commented 7 months ago

Asiantuntijaryhmän päätös 18.3.2024: Lisätään lokiin sisäänkirjautuminen Finnaan.

lmstrand commented 5 months ago

Finna-pluginiin lisätty onnistuneen kirjautumisen yhteydessä lokitus. Virkailijana näkyy Finna-API -tunnus (ps. testeillä nimet on anonymisoitu).

lmstrand commented 5 months ago

Tässä on ongelmana onnistuneen kirjautumisen lokittuminen useampaan kertaan, koska Finnasta kutsutaan asiakkaan get-metodia (jonne lokitustapahtuman koodi on lisätty) useampaan kertaan.

Kati-Helle commented 5 months ago

Helle-testausta. Finnaan kirjautumisesta muodostuu kaksi lokititietoa. Lokitiedon Virkailija-kentässä on Finna-tunnus (nimi ja borrowernumber), ID-tunnus-kentässä on asiakkaan nimi ja borrowernumber.

sksandel commented 5 months ago

Vaski-testillä lokiin tulee Finna-käyttäjän kirjautumisesta 1-2 Login-riviä riippuen siitä mille sivulle käyttäjä kirjautumisessa siirtyy. Lisäksi jokaisesta sivulatauksesta, jossa Finna hakee tietoja Kohasta syntyy Login-rivi (lainat, varaukset, omat tiedot, hakutulos jne).

Rivejä tulee siis paljon, mutta verkkokirjastoon kirjautumisesta ja siellä toimimisesta jäisi jälki.

lmstrand commented 5 months ago

Pitää siirtää tuo lokitus sitten muualle.

lmstrand commented 5 months ago

Testeillä lokitus on siirrettynä asiakkaan tunnusten validointitapahtumaan ennen kuin asiakastiedot palautetaan onnistuneen tunnusten tarkistuksen jälkeen. Asiakas näkyy itse lokitapahtuman tehneenä, mutta tämä on ok.

sksandel commented 5 months ago

Vaski-testillä vaikuttaisi siltä, että edelleen lokirivejä syntyy kirjautumisen lisäksi verkkokirjaston käytöstä kerran minuutissa. Enää emme saaneet aikaiseksi tuplarivejä.

lmstrand commented 5 months ago

Vaaran Finna-testillä ei näyttäisi lokittuvan minuutin välein. Kirjauduin siis vain Finnaan ja annoin sivun seisoa tekemättä mitään.

lmstrand commented 5 months ago

Mutta jos sivustolla käy eri paikoissa, lokia tulee tosiaan n. minuutin välein, mutta ei jokaisella sivulatauksella. Tätä ei voi fiksata meidän päässä.

AnneliO commented 5 months ago

Koha-Suomen palaveri 10.6.2024: Lari kysyy Finna-toimistosta, miksi kirjautumisia tulee niin paljon. Päätetään vastauksen saatuamme, miten edetään.

lmstrand commented 4 months ago

Vastauksena tämä: "Tämä on osa tasapainottelua sen kanssa, kuinka usein tarkistetaan, että käyttäjän kortti on edelleen voimassa."

lmstrand commented 2 months ago

Kehittäjäpalaveri 20.8.2024: Tehdään onnistuneiden kirjautumisten lokituksesta testijakso ja tarkkaillaan, paljonko tuotantoympäristöissä lokia kertyy. Jos määrä on "hallittavissa", säilytetään ominaisuus mukana.

lmstrand commented 2 months ago

Viimeisin commit Revert Log successful credentials validation as login event tiputettu tuotannoista testauksen ajaksi.

lmstrand commented 1 month ago

Revert-commit palautettu ja lokitus poistettu käytöstä.

lmstrand commented 1 month ago

Lokituksen poisjättämisen syynä oli kymmenien ja joidenkin asiakkaiden tapauksissa satojenkin kirjautumislokimerkintöjen syntyminen testijakson aikana. Vaskissa viikon testijakson aikana lokirivejä muodostui n. 100000 riviä yhteensä.

MikkoLiimatainen commented 1 month ago

Eli tämän tiketin asioista auki on vielä tuo asiakkaan omien tietojen katselun lokittaminen. Oliko tämä päätetty jättää asiantuntijaryhmässä toteuttamatta? Ei ole kirjattuna kuin tuo verkkokirjaston lokitus 18.3.2024 muistiossa. Jos tietojen katselun lokitusta ei edistetä, niin tämän tiketin voinee sulkea?