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.

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

Fixed issue with toolbar expecting urls to start with __debug__. #1453

Merged
merged 2 commits into from
Mar 6, 2021
Merged

Fixed issue with toolbar expecting urls to start with __debug__. #1453

merged 2 commits into from
Mar 6, 2021

Commits on Feb 12, 2021

  1. Fixed issue with toolbar expecting urls to start with __debug__. 

    When the history panel was added and we expanded the number of
urls we were instrumenting, there was a check added to exclude
the requests to the toolbar's views. This was done by checking
the path of the request to see if it started with __debug__.
However, the documentation states that is not a requirement.
This change checks the urls to see if the namespace of the
resolved url matches the toolbar's namespace.

This still isn't perfect as another url path could contain the
last namespace of djdt, but this is much less likely than a user
defining the toolbar's urls at a different path than /__debug__/
    @tim-schilling
    tim-schilling committed Feb 12, 2021
    Configuration menu
    Copy the full SHA
    d07c2d5 View commit details
    Browse the repository at this point in the history

  2. Fixed documentation spelling error.

    @tim-schilling
    tim-schilling committed Feb 12, 2021
    Configuration menu
    Copy the full SHA
    135e7ca View commit details
    Browse the repository at this point in the history