Skip to content

Comments

Automatic regeneration of index.html in temp if needed#4

Open
Phyks wants to merge 4 commits intobroncowdd:masterfrom
Phyks:master
Open

Automatic regeneration of index.html in temp if needed#4
Phyks wants to merge 4 commits intobroncowdd:masterfrom
Phyks:master

Conversation

@Phyks
Copy link

@Phyks Phyks commented Dec 21, 2013

Automatically regenerate the index.html file in temp if doesn't exist (this ensures that it is always present, even if accidentally deleted while cleaning temp dir).

  • security update as discussed by e-mail.

* Uniqid generated for each file => never store the file on the server
with its real name.
* Modified the glob function to remove these files.
* Delete the downloaded files as soon as zip archive is generated.
As discussed here :
broncowdd@4a47c06#commitcomment-4929833

Automatically add a .htaccess file in temp/ dir to serve files as plain
text and not executing them.
Forgot about zip files to be served as zip :/
@Phyks
Copy link
Author

Phyks commented Dec 23, 2013

J'ai ajouté deux commits supplémentaires pour résoudre les problèmes de chmod et d'exécution des fichiers téléchargés discutés par e-mail.

@broncowdd
Copy link
Owner

cool, ta solution semble bien mieux , je regarde de plus près et je merge ^^
asap, of course ;)

@broncowdd
Copy link
Owner

au passage, comme je suis une quiche lorraine en ligne de commande github, je ferai le merge à la main, histoire de ne pas perdre les autres ajouts de la 1.4...

@Phyks
Copy link
Author

Phyks commented Dec 23, 2013

Ok ty.

Pour le merge, vu que ça va conflicter, je pense qu'il n'y a pas le choix de toutes façons. Si tu veux, je peux essayer de te proposer une pull request qui merge toute seule...

@broncowdd
Copy link
Owner

t'embête pas ;) je suis dessus ^^ thx

@broncowdd
Copy link
Owner

Curieux, en local , pas de soucis mais sur mon serveur distant, paf erreur 500... pourtant, pas d'erreur levée, pas de pb de droits d'accès... c'est lors de la redirection vers le fichier zip...

@broncowdd
Copy link
Owner

c'est le htaccess qui fait ça...

@Phyks
Copy link
Author

Phyks commented Dec 23, 2013

AllowOverride mis à None sur ton serveur ?

Ou un truc du genre, je pense que c'est qu'il ne prend pas (ou refuse de prendre) le htaccess. Je n'ai aucun problème en local ou sur mon serveur (Debian stable).

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.

2 participants