Déplacement des attachments hors du répertoire public

[dvergeylen]

Ceci permet de laisser les images privées et non accessibles à tous depuis l'extérieur (nécessité d'être loggé)😎

Un petit désagrément (qui m'a pris longtemps à trouver) est qu'il faut précéder a.image() de "/" +, autrement image_tag interprète ceci comme devant être servi comme un asset. (autrement ceci génèrerait une erreur car les images ne sont pas des assets mais des fichiers envoyés via send_file par un controller, après vérification d'accès).

Il faut maintenant écrire un script de migration des uploads actuels vers le nouveau répertoire.

Pour chaque attachement, le path est le suivant:

:rails_root/uploaded_files/:attachement_id/:style.:extension

Ceci nécessite d'être testé avant d'être déployé (marche chez moi depuis un fresh clone).

Fix #75

[svergeylen]

mouais... j'ai cliqué sur le bouton et maintenant ?

[dvergeylen]

Et bien dev contient maintenant les modifications proposées par paperclip comme si j'avais commité dans dev.

Si les tests n'avaient pas été concluants (tu as testé... n'est-ce pas? 😇) j'aurais pu continuer à commiter dans paperclip jusqu'à ce que ça soit ok et tu n'aurais fait le merge que lorsque tout aurait été bon.

Ça permet de n'avoir qu'une seule branche par problème (et/ou par développeur), ce qui évite des commits successifs mais qui parlent de problèmes différents. Je sais qu'ici c'est un peu overkill mais ça vaut la peine de s'habituer au flow pour des projets où ça n'est pas directement critique.

Un excellent tutoriel ici: https://github.com/susam/gitpr

[dvergeylen]

:warning: Avant qu'on ne déploie les modifications de dev vers master, il faudra migrer les images uploadées

[svergeylen]

Et voilà j'ai fait une belle tache rake pour déplacer les uploads :-) rake files:move_uploads Je ne vois pas comment lier cela au commit mais peu importe : 417f018ce6f3429b6d4c188a8cdc1cd9a7e59e8d

Tu peux vérifier mon beau script ? ;-)

[svergeylen]

ha bein il suffisait de coller le numéro de commit complet - c'est beau - même s'il n'affiche qu'une partie du numéro de commit (c'est sans doute pour faire plus moderne !) :-)

[svergeylen]

Ha bein non, j'ai cliqué sur le bouton merge pour avoir tes modifs, mais j'ai pas encore testé, c'est que je vais faire maintenant ;-)

[dvergeylen]

C'est une belle tâche rake en effet, super ! :+1:

[dvergeylen]

Ooh :disappointed: , pourtant c'est si simple, il suffit de faire git clone de ma branche sur mon répertoire...

[svergeylen]

OK, tu me montreras... parce que le clone c'est le truc qu'on fait au début non ? ;-)

et j'ai oublié de fair eun rm -r de l'ancien dossier ettachements dans public.. mais on pourra le faire à la main si tout marche bien non ??

J'ai pas regardé si les droits sur les fichiers sont bons, mais a priori, si c'est le même user qui fait la tache rake et celui qui utilise le site web, ca derait rouler... ;-)

[dvergeylen]

Si tu supprimes les attachs maintenant, plus aucun post de va marcher puisque dev n'est pas rapatrié dans master ni encore déployé sur le serveur...

L'implem actuelle sur le serveur s'attend encore à trouver les attachs sur l'ancien PATH. Ou alors tu déploies pour tester? :scream:

[svergeylen]

non non, on s'est pas compris.. j'ai oublié de supprimer les dossiers à la fin du script... mais c'est pas grave du tout... il faut maintenant revoir tout le collector pour faire fonctionner les tag ! Ya du boulot car ya plein de vues qui s'affichent plus et il y des fonctionnalités qui existaient avant qui ne sont plus accessibles, etc... justement, je m'y remets tout doucement... ;-)