Fix bug #2 - Githubissues

rancavil / django-py3-openshift-quickstart

Django 1.6 git repository for OpenShift with Python 3.3

33 stars 20 forks source link

Fix bug #2 #3

Open Fale opened 10 years ago

Fale commented 10 years ago

As declared, if you merge this pull request, you can safely close bug report #2 since this is the actual code suggested in that bug pulll-requested

Enrices commented 9 years ago

Very useful fix but I disagree with the change in manage.py.

Firstly, this change should be done in settings.py (using for example BASE_DIR) Secondly, this is important to wrap the new path with os.path.normpath(). If not, some OS will not recognize the URL foo/../bar and the system will crash.

I will suggest another fix. (I cannot change this one as it is not pulled yet)