entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 38 Releases Activity

data sources: fix caching (#73045) #12

Merged
fpeters merged 1 commits from wip/73045-data-source-cache into main 2023-01-05 19:49:25 +01:00
Conversation 0 Commits 1 Files Changed 3 +70 -8
fpeters commented 2023-01-05 19:29:15 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-01-05 19:29:15 +01:00
gitea-wip/wcs/pipeline/pr-main This commit looks good Details
01eff2566f data sources: fix caching (#73045)
fpeters reviewed 2023-01-05 19:32:07 +01:00
wcs/api.py
@ -1195,2 +1195,3 @@
cache_duration = named_data_source.cache_duration
if named_data_source.cache_duration:
cache_duration = int(named_data_source.cache_duration)
url = info['url']
fpeters commented 2023-01-05 19:30:29 +01:00
Author
Owner
Copy Link

C'est le premier problème; sans ça au moment du cache.set on a

TypeError: '>' not supported between instances of 'str' and 'int'

C'est le premier problème; sans ça au moment du cache.set on a > TypeError: '>' not supported between instances of 'str' and 'int'
wcs/api.py
@ -1196,4 +1197,3 @@
url = info['url']
url += urllib.parse.quote(get_request().form.get('q', ''))
url = sign_url_auto_orig(url)
get_response().set_content_type('application/json')
fpeters commented 2023-01-05 19:31:35 +01:00
Author
Owner
Copy Link

Deuxième problème on signe ici, et donc on demanderait à chaque fois au cache une URL différente.

Deuxième problème on signe ici, et donc on demanderait à chaque fois au cache une URL différente.
wcs/data_sources.py
@ -263,1 +261,4 @@
geojson = data_source.get('type') == 'geojson'
error_summary = None
try:
fpeters commented 2023-01-05 19:31:55 +01:00
Author
Owner
Copy Link

Troisième problème, similaire, il faut signer après avoir regardé le cache. (le bout quoté par gitea n'est pas clair, il s'agit en fait d'un bout plus gros, déplacé du début de la fonction à après la vérification du cache).

Troisième problème, similaire, il faut signer après avoir regardé le cache. (le bout quoté par gitea n'est pas clair, il s'agit en fait d'un bout plus gros, déplacé du début de la fonction à après la vérification du cache).
Ghost approved these changes 2023-01-05 19:37:29 +01:00
Ghost left a comment
First-time contributor
Copy Link

J'avais raté #39723 ... et donc oui, bien vu, merci.

J'avais raté #39723 ... et donc oui, bien vu, merci.
fpeters merged commit 01eff2566f into main 2023-01-05 19:49:25 +01:00
fpeters deleted branch wip/73045-data-source-cache 2023-01-05 19:49:25 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Ghost
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#12
No description provided.
Delete Branch "wip/73045-data-source-cache"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?