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.

Sign up for GitHub

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

Jump to bottom

Correct WSGI adapter handling of root path. #446

Merged

carltongibson merged 1 commit into main from wsgi-adapter-root-path

Mar 15, 2024

Member

carltongibson commented Mar 14, 2024

Fixes #442


          Correct WSGI adapter handling of root path.

939fc9e

Fixes #442

carltongibson commented

View reviewed changes

asgiref/wsgi.py

+                      script_name = scope.get("root_path", "").encode("utf8").decode("latin1")
+                      path_info = scope["path"].encode("utf8").decode("latin1")
+                      if path_info.startswith(script_name):
+                          path_info = path_info[len(script_name) :]

Member Author

carltongibson Mar 14, 2024

The setup.cfg change was required by this.

% flake8 asgiref/wsgi.py 
asgiref/wsgi.py:60:51: E203 whitespace before ':'

andrewgodwin approved these changes

View reviewed changes

carltongibson merged commit 4209b6c into main

6 checks passed

carltongibson deleted the wsgi-adapter-root-path branch

March 15, 2024 16:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment