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

[API] Gestion des fichiers #3551

Open
wants to merge 3 commits into
base: feature/3424-api-amelioration
Choose a base branch
from

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Jan 14, 2025

Ticket

#3525

Description

L'absence du nom de fichier dans l'url public du fichier rend la ressource téléchargeable pas très exploitable et peut compliquer le traitement par des langage de programmation.

Démo effectué avec un client curl

curl -O -J https://histologe.beta.gouv.fr/show/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

On se retrouve avec l'uuid comme nom de fichier et sans extension.

Pour rendre la ressource exploitable, le plus simple est de passer par l'en-tête HTTP Content-Disposition

Changements apportés

  • Mise à jour FileController
  • Mise à jour documentation API pour les fichiers

Pré-requis

Tests

image

- L'option -O est pour télécharger l'image
- L'option -J est pour utiliser le nom spécifié dans l'en-tête http `Content-disposition`

@sfinx13 sfinx13 changed the title Feature/3525 api file [API] Gestion des fichiers Jan 14, 2025
@sfinx13 sfinx13 force-pushed the feature/3424-api-amelioration branch from acd2a96 to 8440a57 Compare January 15, 2025 09:36
@sfinx13 sfinx13 force-pushed the feature/3525-api-file branch from db52e02 to 34a798f Compare January 15, 2025 10:02
@sfinx13 sfinx13 force-pushed the feature/3525-api-file branch from 34a798f to 43a1085 Compare January 15, 2025 10:09
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

Successfully merging this pull request may close these issues.

1 participant