search

benjamintd / attestation.page

Attestation en un clic
https://attestation.page
MIT License
23 stars 1 forks source link

Mauvais nom de fichier lors du téléchargement #12

Open cycha opened 3 years ago

cycha commented 3 years ago

Sur Chrome Android, le nom du fichier est un UUID et n'a rien à voir avec ce qu'il y a dans le code. Je suis pas assez bon en React pour dire pourquoi cela ne suit pas le code.

https://github.com/benjamintd/attestation.page/blob/99f9060d7598916411b639f8dd2781683091d52e/pages/index.jsx#L85

benjamintd commented 3 years ago

Hello @cycha,

J'ai mis une condition particulière sur Chrome pour que le document s'ouvre dans un nouvel onglet avec le lecteur pdf intégré du navigateur, plutôt que de télécharger le fichier.

Cela se passe ici: https://github.com/benjamintd/attestation.page/blob/a5584f094ce26df7dec9ea5a3b215231e3d97b57/lib/util.js#L54-L68

Il semble que je ne puisse malheureusement pas contrôler le nom de l'URL dans ce cas, mais je trouve préférable d'avoir l'attestation ouverte directement par Chrome plutôt que d'avoir un fichier supplémentaire dans les téléchargements de mon appareil.

Qu'en penses-tu ?

cycha commented 3 years ago

Bonjour @benjamintd,

Ah pardon en effet je n'étais pas rentré dans openBlob(), la logique voudrait que l'on renomme le blob pour enlever le UUID du nom de fichier par défaut mais après un petit tour sur stackoverflow il semblerait que cela soit impossible => https://stackoverflow.com/questions/41947735/custom-name-for-blob-url/51958546

Donc oui il me semble que c'est plus pratique d'avoir l'ouverture automatique qu'un joli nom.

Merci pour la réponse.