kaleidoscopique / Cakebox

Seedbox clef-en-main avec streaming vidéo intégré
25 stars 10 forks source link

Faire un vrai bouton telecharger, affichage correct du chemin #10

Closed oursours closed 11 years ago

oursours commented 11 years ago

Je ne sais pas bien le moyen, mais pour le moment le bouton telecharger force juste la lecture de la video ... il faudrait 2 boutons bien distincts, un qui propose le telechargement et un qui force la lecture.

Le chemin du fichier est afficher en requete web aussi et pas en normal.

kaleidoscopique commented 11 years ago

Merci d'insister là dessus, je voulais m'en occuper mais c'était passé tout en bas de ma todo list. Je le fais remonter.

tuxity commented 11 years ago

Je viens de regarder et faire quelques tests pour forcer le téléchargement et, je trouve que le temps ou je clique sur l'image et le temps où la popup de firefox apparait est ... trop long (par rapport a un clique droit, enregistrer la cible du lien sous)

J'ai joué avec header() et readfile() de php, j'ai pas encore trouvé une autre alternative :/

kaleidoscopique commented 11 years ago

J'aurais pas fait autrement que readfile. Ça met combien de temps au clic ?

tuxity commented 11 years ago

Sur un fichier de 500mo je dirais environ 15 secondes

kaleidoscopique commented 11 years ago

Est-ce que dans tes header tu as mis "header ("Content-type: octet/stream");" ?

tuxity commented 11 years ago

Hmmm je me souviens plus, il me semble oui

Magicalex commented 11 years ago

salut,

j'ai fais deux trois recherches, et j'ai découvert qu'on peut forcer le téléchargement en html avec un attribut download (c'est de l'html 5) http://www.creativejuiz.fr/blog/tutoriels/html5-attribut-download-pour-forcer-telechargement sûrement pas compatible avec tous les navigateurs Ou même avec le serveur web avec un .htaccess

edit:

<a href="<?php echo DOWNLOAD_LINK.$file; ?>" download="<?php echo ustr_replace(LOCAL_DL_PATH."/","",$file ); ?>" >
kaleidoscopique commented 11 years ago

Sympa ce truc download. Les gens qui utilisent Cakebox sont quand même un peu débrouillards et ne trainent pas des nav trop pourris. Je pense que c'est une bonne façon de faire.

Magicalex commented 11 years ago

je viens de tester ça marche vraiment bien ps : voir mon message précédent

tuxity commented 11 years ago

Je connaissais pas cet attribut, je vote pour. Sinon le htaccess c'est un peu chiant :x

Magicalex commented 11 years ago

Le problème s'est que il n y a que chrome qui support cette attribut. firefox (v17 dans mon cas), opera et ie ça ne fonctionne pas

Vinky41 commented 11 years ago

Je pense que c'est le bon lieu pour parler du chemin justement.

J'ai remarqué que lorsque les liens contiennent login/mdp dans l'url.

Je pense que le htaccess oblige cela, mais niveau sécurité c'est pas terrible tout de même :( Alors c'est toujours pareil, il est conseillé de se connecter en VPN en amont, mais bon il suffit qu'un pote regarde ce que l'on télécharge pour que login/mdp soit récupérer très facilement :(

Après, techniquement, je ne vois aps du tout comment empêcher ça. A titre perso. j'ai déjà fait en sorte dans le htaccess qu'en VPN et chez moi, aucun mdp n'est demandé (ça évite que dans les deux cas une personne à côté de moi lise le login & mdp, mais ce n'est pas une solution ultime je trouve.

tuxity commented 11 years ago

Pour en revenir a l'histoire, je pense qu'on peut mettre l'attribut download pour les chrome users, mais on ne devrait pas forcer le téléchargement via script/htaccess. Forcer un utilisateur c'est pas super mais en plus les méthodes actuelles sont lentes sur certains fichiers, donc c'est limite + rapide de faire un clique droit -> copier l'url

kaleidoscopique commented 11 years ago

Faut-il faire un test pour afficher ou non l'attribut pour les chrome user ou on peut le mettre pour tout le monde et il sera ignoré des autres nav ?

tuxity commented 11 years ago

na tu peux le mettre, les autres nav vont l'ignorer et "un jour" le traiter :p

kaleidoscopique commented 11 years ago

C'est pushé.