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

Task - "Last-Modified" response header and value for each feature #72

Open
1 of 4 tasks
Tracked by #12
benoitdm-oslandia opened this issue Oct 21, 2022 · 5 comments
Open
1 of 4 tasks
Tracked by #12

Comments

@benoitdm-oslandia
Copy link
Owner

In GitLab by @nrevelant on Oct 21, 2022, 10:27

Actuellement une valeur de last-modified est stockée dans la structure ajoutée au cache lors du traitement du chargement d'une feature.

  • Cette valeur devrait-elle être stockée de manière pérenne (en base) ou seulement dans le cache des etag lors du fonctionnement du service ?
  • La création de cette date devrait-elle être déléguée à la base de donnée (intégrée aux triggers lors des CREATE, UPDATE) ? (Actuellement la date est générée à la volée par le service en remplissant le cache des etag)
  • La suppression de cette valeur doit-elle être intégrée au trigger effectuant les DELETE de feature en base (que la date stockée ou pas en base) ?

@benoitdm-oslandia ( et un peu @azarz ;-) ) ?

  • Formaliser la valeur exacte à générer en tant que Last-Modified. A priori <day-name>, <day> <month> <year> <hour>:<minute>:<second> GMT quelle que soit la zone locale @benoitdm-oslandia ?
    -> Forme HTTP de la date avec les headers If-Modified-Since et If-Unmodified-Since :
    <day-name>, <day> <month> <year> <hour>:<minute>:<second> GMT

Example:
If-Modified-Since: Wed, 21 Oct 2015 07:28:00 GMT

  • Générer/récupérer la date de dernière modification d'une feature suite à chaque modification POST/PUT/PATCH (trigger)
  • Renvoyer au client cette valeur dans un header "Last-Modified" dans le cas d'un GET sans header
  • Renvoyer au client cette valeur dans un header "Last-Modified" dans les éventuels autres cas à identifier de manière exhaustive (headers)
@benoitdm-oslandia
Copy link
Owner Author

In GitLab by @nrevelant on Oct 26, 2022, 15:20

marked this issue as related to #81

@benoitdm-oslandia
Copy link
Owner Author

In GitLab by @nrevelant on Oct 26, 2022, 15:21

marked this issue as related to #80

@benoitdm-oslandia
Copy link
Owner Author

In GitLab by @azarz on Oct 27, 2022, 09:32

Si la valeur du timestamp est générée par les triggers, seules les modifications qui ont lieu alors que PGFeatureServ tourne sont prises en compte : de fait les intégrer dans le cache est aussi une possibilité qui ne change fonctionnellement rien.

De plus, en partant du principe que PGFeatureServ se branche sur des BDD "déjà existantes", je ne pense pas qu'il ait à modifier la "structure" de la BDD, et donc je pense qu'ajouter des colonnes n'est pas génial.

Et en fait je viens de chercher un truc, pas besoin d'ajouter un champ dans tous les cas car l'information est récupérable à partir du xmin via pg_xact_commit_timestamp().

@benoitdm-oslandia
Copy link
Owner Author

marked the task Formaliser la valeur exacte à générer en tant que Last-Modified. A priori <day-name>, <day> <month> <year> <hour>:<minute>:<second> GMT quelle que soit la zone locale @benoitdm-oslandia ? as completed

@benoitdm-oslandia
Copy link
Owner Author

marked this issue as related to #10

@benoitdm-oslandia benoitdm-oslandia added this to the 04_SPRINT milestone Nov 10, 2022
@benoitdm-oslandia benoitdm-oslandia removed this from the 04_SPRINT milestone Dec 5, 2022
@nick-rv nick-rv changed the title Task - Header de réponse "Last-Modified" et valeur pour chaque feature Task - "Last-Modified" response header and value for each feature Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant