fix file permissions after each sync or deployment

evolution / wordpress

Rapidly create, develop, & deploy WordPress across multiple environments.

MIT License

200 stars 18 forks source link

fix file permissions after each sync or deployment #140

Closed jalevin closed 8 years ago

jalevin commented 8 years ago

Various parts of the deployment messed with file permissions that caused problems when uploading files using gravity forms. I this commit simply runs evolve:permissions after deployment or whenever files are synced with evolve:up

EvanK commented 8 years ago

Thanks for this, I can get it rolled into the next release!

jalevin commented 8 years ago

Original patch didn't fix the permissions of the upload folder. Changed that to explicitly set those permissions.

jalevin commented 8 years ago

bump. can you take another look at this?

EvanK commented 8 years ago

I'm not certain what permissions issue you were encountering, but this looks like a simple enough change:

enforces www-data group as well as deploy owner, for all wp-content subdirectories
enforces group writable permission for uploads subdirectories

We can get this merged 👍

jalevin commented 8 years ago

Specifically Gravity forms was running into permissions errors when uploading files and when I started investigating permissions, it looked like any plugins that upload files would encounter this issue.

Thanks!!

EvanK commented 8 years ago

Tagged & released as v1.4.4