journal de suivi des consultations (#51974) #22

fpeters · 2023-01-10T11:14:40+01:00

fpeters commented

2023-01-10 11:14:40 +01:00

Dans 0001 l'ajout du modèle et d'une fonction générale (audit) ajoutée aux endroits adéquats, façon audit('listing', obj=self.formdef). Il y a deux points qui pourraient se discuter : faire la requête au listing avec ?ajax=true ne loggue pas l'accès (pour ne pas avoir le journal rempli à chaque rafraichissement) et l'accès aux vignettes des images n'est pas loggué non plus (dans l'idée que l'accès à la demande est loggué et suffit). Je vais porter cette question dans le redmine pour la visibiliser.

Dans 0002 la vue backoffice, elle est accessible uniquement aux personnes "admin", elle permet de filtrer sur la date, le formulaire/modèle de fiche, ou une demande/fiche individuelle, et l'utilisateur ayant fait l'action. Comme l'ajout d'événements est continu la pagination est différente et se fait juste avec des boutons (le plus récent, plus récent, plus ancien, le plus ancien), pas de numéros de page.

Dans 0003 l'expiration, après deux ans par défaut (comme pour le journal d'authentic), rien de particulier.

Dans 0001 l'ajout du modèle et d'une fonction générale (audit) ajoutée aux endroits adéquats, façon `audit('listing', obj=self.formdef)`. Il y a deux points qui pourraient se discuter : faire la requête au listing avec ?ajax=true ne loggue pas l'accès (pour ne pas avoir le journal rempli à chaque rafraichissement) et l'accès aux vignettes des images n'est pas loggué non plus (dans l'idée que l'accès à la demande est loggué et suffit). Je vais porter cette question dans le redmine pour la visibiliser. Dans 0002 la vue backoffice, elle est accessible uniquement aux personnes "admin", elle permet de filtrer sur la date, le formulaire/modèle de fiche, ou une demande/fiche individuelle, et l'utilisateur ayant fait l'action. Comme l'ajout d'événements est continu la pagination est différente et se fait juste avec des boutons (le plus récent, plus récent, plus ancien, le plus ancien), pas de numéros de page. Dans 0003 l'expiration, après deux ans par défaut (comme pour le journal d'authentic), rien de particulier.

fpeters force-pushed wip/51974-audit from 45b35be07c to 09f6a2c90a

2023-01-11 19:42:40 +01:00

Compare

lguerin requested changes 2023-01-12 09:27:09 +01:00

wcs/sql.py Outdated

						
				@ -4574,0 +4673,4 @@

				        conn, cur = get_connection_and_cursor()

				        sql_statement = 'SELECT id FROM audit ORDER BY id LIMIT 1'

				        cur.execute(sql_statement)

				        first_id = cur.fetchall()[0][0]

lguerin commented

2023-01-12 09:16:00 +01:00

Ca n'arrivera qu'une fois, mais en local après migration et consultation de la page d'audit, j'ai une trace:

Exception:
  type = '<class 'IndexError'>', value = 'list index out of range'

Stack trace (most recent call first):
  File "/home/lguerin/src/wcs/wcs/sql.py", line 4676, in get_first_id
  4674         sql_statement = 'SELECT id FROM audit ORDER BY id LIMIT 1'
  4675         cur.execute(sql_statement)
> 4676         first_id = cur.fetchall()[0][0]
  4677         conn.commit()
  4678         cur.close()

Ca n'arrivera qu'une fois, mais en local après migration et consultation de la page d'audit, j'ai une trace: ``` Exception: type = '<class 'IndexError'>', value = 'list index out of range' Stack trace (most recent call first): File "/home/lguerin/src/wcs/wcs/sql.py", line 4676, in get_first_id 4674 sql_statement = 'SELECT id FROM audit ORDER BY id LIMIT 1' 4675 cur.execute(sql_statement) > 4676 first_id = cur.fetchall()[0][0] 4677 conn.commit() 4678 cur.close() ```

fpeters commented

2023-01-13 14:07:06 +01:00

Pris en compte pour désormais faire

+        try:
+            first_id = cur.fetchall()[0][0]
+        except IndexError:
+            first_id = 0

Pris en compte pour désormais faire ``` + try: + first_id = cur.fetchall()[0][0] + except IndexError: + first_id = 0 ```

lguerin marked this conversation as resolved

fpeters force-pushed wip/51974-audit from 09f6a2c90a to 032f383bec

2023-01-13 12:55:47 +01:00

Compare

fpeters force-pushed wip/51974-audit from 032f383bec to f03eed67c0

2023-01-13 12:56:45 +01:00

Compare

lguerin approved these changes 2023-01-13 14:09:21 +01:00

fpeters commented

2023-01-13 14:10:12 +01:00

Aussi, j'ai modifié pour remplacer « par ≪ (qui est le symbole qu'on utilise déjà pour la navigation entre snapshots) (et » par ≫ bien sûr).

Plus important par rapport à ma question "à chaque rafraichissement" j'ai modifié pour que ça soit loggué à chaque fois, mais en ajoutant l'info en extra_data, si jamais on veut un jour filtrer.

J'ai aussi modifié le stockage dans l'attribut url; il ne reprend/répète plus le nom de domaine, uniquement le chemin + la query string qui manquait.

Et jai actualisé le numéro de migration.

Aussi, j'ai modifié pour remplacer « par ≪ (qui est le symbole qu'on utilise déjà pour la navigation entre snapshots) (et » par ≫ bien sûr). Plus important par rapport à ma question "à chaque rafraichissement" j'ai modifié pour que ça soit loggué à chaque fois, mais en ajoutant l'info en extra_data, si jamais on veut un jour filtrer. J'ai aussi modifié le stockage dans l'attribut url; il ne reprend/répète plus le nom de domaine, uniquement le chemin + la query string qui manquait. Et jai actualisé le numéro de migration.

fpeters merged commit dc4fe98294 into main

2023-01-13 14:47:24 +01:00

fpeters deleted branch wip/51974-audit

2023-01-13 14:47:25 +01:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#22