Opened 17 years ago
Closed 16 years ago
#6647 closed (fixed)
WSGIRequestHandler dies on "OPTIONS *" request
Reported by: | nospampleasethanks | Owned by: | nobody |
---|---|---|---|
Component: | HTTP handling | Version: | dev |
Severity: | Keywords: | wsgi http server error | |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
An exception is raised by the WSGIRequestHandler:
Exception happened during processing of request from ('127.0.0.1', 60148) Traceback (most recent call last): File "SocketServer.py", line 222, in handle_request self.process_request(request, client_address) File "SocketServer.py", line 241, in process_request self.finish_request(request, client_address) File "SocketServer.py", line 254, in finish_request self.RequestHandlerClass(request, client_address, self) File "/usr/lib/python2.5/site-packages/django/core/servers/basehttp.py", line 554, in __init__ BaseHTTPRequestHandler.__init__(self, *args, **kwargs) File "SocketServer.py", line 522, in __init__ self.handle() File "/usr/lib/python2.5/site-packages/django/core/servers/basehttp.py", line 595, in handle if not self.parse_request(): # An error code has been sent, just exit File "/usr/lib/python2.5/BaseHTTPServer.py", line 279, in parse_request self.send_error(400, "Bad request syntax (%r)" % requestline) File "/usr/lib/python2.5/BaseHTTPServer.py", line 343, in send_error self.log_error("code %d, message %s", code, message) File "/usr/lib/python2.5/BaseHTTPServer.py", line 414, in log_error self.log_message(*args) File "/usr/lib/python2.5/site-packages/django/core/servers/basehttp.py", line 603, in log_message if self.path.startswith(self.admin_media_prefix) or self.path == '/favicon.ico': AttributeError: WSGIRequestHandler instance has no attribute 'path'
when an HTTP client makes the following request:
OPTIONS * / HTTP/1.1
Django raises a 500 Internal Server Error. Should it return a 400 Bad Request instead?
A patch defining a dummy self.path is attached.
Attachments (1)
Change History (4)
by , 17 years ago
Attachment: | self_path_dummy.patch added |
---|
comment:1 by , 17 years ago
Update: The line is "OPTIONS * HTTP/1.1" (thanks to Magus-, from #django fame).
comment:2 by , 16 years ago
Triage Stage: | Unreviewed → Ready for checkin |
---|
comment:3 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
self.path dummy definition patch