Skip to content

Commit

Permalink
Upgrade: revert change to default LOCAL_UPLOADS_DIR in prod settings.
Browse files Browse the repository at this point in the history
The main purpose of the "var" convention is to make it easy to write stuff
inside of our git repo when running a dev instance, and then "var" gets
excluded from checkins. For production, that's not as much of a concern.
For upgrades we don't want to be changing the directory around and confusing
matters, especially with the extra moving part of nginx configs (which have
their own issues in terms of being overwritten by accident when admins go to
S3).
  • Loading branch information
showell authored and rishig committed Sep 2, 2016
1 parent 0120ff5 commit f6975f9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion zproject/prod_settings_template.py
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@
# https://github.com/zulip/zulip/issues/291 for discussion of a better
# solution that won't be automatically reverted by the Zulip upgrade
# script), and then restart nginx.
LOCAL_UPLOADS_DIR = "/home/zulip/var/uploads"
LOCAL_UPLOADS_DIR = "/home/zulip/uploads"
#S3_AUTH_UPLOADS_BUCKET = ""
#S3_AVATAR_BUCKET = ""

Expand Down

0 comments on commit f6975f9

Please sign in to comment.