Incorrect message on 404 debug page for an empty request path

[Mariusz Felisiak]

If empty request path r'^$' is not in URL patterns, then message on 404 debug page is incorrect:

Using the URLconf defined in test_app.urls, Django tried these URL patterns, in this order:

    ^admin/
    ^__debug__/

The current URL, , didn't match any of these.

it should be (without double comma):

Using the URLconf defined in test_app.urls, Django tried these URL patterns, in this order:

    ^admin/
    ^__debug__/

The current URL, didn't match any of these.

[Robert Roskam]

Your fix for this case (​https://github.com/django/django/pull/7420/files) is questionable, IMO.

If your goal is to help the user determine the URL that they attempted to resolve, the double comma at least helps visually enclose the space that's trying to be resolved.

I would suspect that this case is most common for new Django users. If this case is difficult enough to communicate, it may be worth it to come up with a better overall message.

[Tim Graham]

I noticed this issue at one point and debated whether or not it was worth filing a ticket. It could be confusing, I guess.

[Mariusz Felisiak]

Maybe sth like this:

<p>
  {% if request_path %}
    The current URL, <code>{{ request_path|escape }}</code>,
  {% else %}
    The empty URL
  {% endif %}didn't match any of these.
</p>

will be better message.

[Tim Graham <timograham@…>]

In c7f86d3e:

Fixed #27373 -- Corrected 404 debug page message for an empty request path.