Opened 5 years ago
Closed 5 years ago
#31166 closed Cleanup/optimization (fixed)
Provide context for ImproperlyConfigured exceptions in URL resolver.
Reported by: | Ram Rachum | Owned by: | Ram Rachum |
---|---|---|---|
Component: | Core (URLs) | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Shai Berger | 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
The patch is ready here: https://github.com/django/django/pull/12263
This will make it have a text of "this exception is the direct result of" instead of "During handling of the above exception, another exception occurred"
This is more accurate for the case of this exception.
If this change will be merged, there are a couple more place in this module where it can be added.
When it happened to me a few weeks ago, it was due to a circular import, which I then fixed. Now of course, when I tried to reproduce the circular import to show you, I couldn't. But I could achieve the same thing by renaming urlpatterns
to xurlpatterns
, so it wouldn't be available. This is the error I get:
Exception in thread django-main-thread: Traceback (most recent call last): File "C:\Program Files\Python38\lib\site-packages\django\urls\resolvers.py", line 590, in url_patterns iter(patterns) TypeError: 'module' object is not iterable During handling of the above exception, another exception occurred: Traceback (most recent call last): File "C:\Program Files\Python38\lib\threading.py", line 932, in _bootstrap_inner self.run() File "C:\Program Files\Python38\lib\threading.py", line 870, in run self._target(*self._args, **self._kwargs) File "C:\Program Files\Python38\lib\site-packages\django\utils\autoreload.py", line 53, in wrapper fn(*args, **kwargs) File "C:\Program Files\Python38\lib\site-packages\django\core\management\commands\runserver.py", line 117, in inner_run self.check(display_num_errors=True) File "C:\Program Files\Python38\lib\site-packages\django\core\management\base.py", line 392, in check all_issues = self._run_checks( File "C:\Program Files\Python38\lib\site-packages\django\core\management\base.py", line 382, in _run_checks return checks.run_checks(**kwargs) File "C:\Program Files\Python38\lib\site-packages\django\core\checks\registry.py", line 72, in run_checks new_errors = check(app_configs=app_configs) File "C:\Program Files\Python38\lib\site-packages\django\core\checks\urls.py", line 13, in check_url_config return check_resolver(resolver) File "C:\Program Files\Python38\lib\site-packages\django\core\checks\urls.py", line 23, in check_resolver return check_method() File "C:\Program Files\Python38\lib\site-packages\django\urls\resolvers.py", line 408, in check messages.extend(check_resolver(pattern)) File "C:\Program Files\Python38\lib\site-packages\django\core\checks\urls.py", line 23, in check_resolver return check_method() File "C:\Program Files\Python38\lib\site-packages\django\urls\resolvers.py", line 407, in check for pattern in self.url_patterns: File "C:\Program Files\Python38\lib\site-packages\django\utils\functional.py", line 48, in __get__ res = instance.__dict__[self.name] = self.func(instance) File "C:\Program Files\Python38\lib\site-packages\django\urls\resolvers.py", line 597, in url_patterns raise ImproperlyConfigured(msg.format(name=self.urlconf_name)) django.core.exceptions.ImproperlyConfigured: The included URLconf '<module 'barb.urls' from 'C:\\Users\\Administrator\\Desktop\\foof\\barb\\urls.py'>' does not appear to have any patterns in it. If you see valid patterns in the file then the issue is probably caused by a circular import.
The bad part is During handling of the above exception, another exception occurred
.
After my fix, it looks like this:
Exception in thread django-main-thread: Traceback (most recent call last): File "C:\Program Files\Python38\lib\site-packages\django\urls\resolvers.py", line 590, in url_patterns iter(patterns) TypeError: 'module' object is not iterable The above exception was the direct cause of the following exception: Traceback (most recent call last): File "C:\Program Files\Python38\lib\threading.py", line 932, in _bootstrap_inner self.run() File "C:\Program Files\Python38\lib\threading.py", line 870, in run self._target(*self._args, **self._kwargs) File "C:\Program Files\Python38\lib\site-packages\django\utils\autoreload.py", line 53, in wrapper fn(*args, **kwargs) File "C:\Program Files\Python38\lib\site-packages\django\core\management\commands\runserver.py", line 117, in inner_run self.check(display_num_errors=True) File "C:\Program Files\Python38\lib\site-packages\django\core\management\base.py", line 392, in check all_issues = self._run_checks( File "C:\Program Files\Python38\lib\site-packages\django\core\management\base.py", line 382, in _run_checks return checks.run_checks(**kwargs) File "C:\Program Files\Python38\lib\site-packages\django\core\checks\registry.py", line 72, in run_checks new_errors = check(app_configs=app_configs) File "C:\Program Files\Python38\lib\site-packages\django\core\checks\urls.py", line 13, in check_url_config return check_resolver(resolver) File "C:\Program Files\Python38\lib\site-packages\django\core\checks\urls.py", line 23, in check_resolver return check_method() File "C:\Program Files\Python38\lib\site-packages\django\urls\resolvers.py", line 408, in check messages.extend(check_resolver(pattern)) File "C:\Program Files\Python38\lib\site-packages\django\core\checks\urls.py", line 23, in check_resolver return check_method() File "C:\Program Files\Python38\lib\site-packages\django\urls\resolvers.py", line 407, in check for pattern in self.url_patterns: File "C:\Program Files\Python38\lib\site-packages\django\utils\functional.py", line 48, in __get__ res = instance.__dict__[self.name] = self.func(instance) File "C:\Program Files\Python38\lib\site-packages\django\urls\resolvers.py", line 597, in url_patterns raise ImproperlyConfigured(msg.format(name=self.urlconf_name)) from type_error # Ram hack, remove django.core.exceptions.ImproperlyConfigured: The included URLconf '<module 'barb.urls' from 'C:\\Users\\Administrator\\Desktop\\foof\\barb\\urls.py'>' does not appear to have any patterns in it. If you see valid patterns in the file then the issue is probably caused by a circular import.
Now the text is corrected to The above exception was the direct cause of the following exception
Change History (4)
comment:1 by , 5 years ago
Summary: | Fix context for ImproperlyConfigured error → Provide context for ImproperlyConfigured exceptions in URL resolver. |
---|---|
Triage Stage: | Unreviewed → Accepted |
Type: | Bug → Cleanup/optimization |
comment:2 by , 5 years ago
Owner: | changed from | to
---|
comment:3 by , 5 years ago
Cc: | added |
---|---|
Triage Stage: | Accepted → Ready for checkin |
Hi Ram. Thanks for the report.
I'm inclined to accept this as a cleanup.
OK. Can we do the whole module at once?
Notes on the PR: