Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dudel.wsgi: correctly find real path of the rest of the code #133

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dario23
Copy link

@dario23 dario23 commented Apr 21, 2017

previously, the abspath() of server.wsgi was added to the python import
path. abspath() however only cleans up the given path string, with no
filesystem interaction like resolving symlinks etc.

some webservers (like apache2) require .wsgi files to be in an
accessible location (like webroot), where for security you might not
want to put your whole code, thus encouraging the use of symlinks.

previously, the abspath() of server.wsgi was added to the python import
path. abspath() however only cleans up the given path string, with no
filesystem interaction like resolving symlinks etc.

some webservers (like apache2) require .wsgi files to be in an
accessible location (like webroot), where for security you might not
want to put your whole code, thus encouraging the use of symlinks.
i couldn't test this yet, so.. use with care
before, the code would just fail on the unique constraint for
user.username
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.

1 participant