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.

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

Fix static urls with yarl 1.9+ #479

Merged
merged 6 commits into from
Sep 13, 2023
Merged

Conversation

14droplets
Copy link
Contributor

What do these changes do?

This changeset resolves #478 which is about broken debug toolbar static file urls with recent versions of yarl dependency.

Are there changes in behavior for the user?

Nothing is changed for the users

Related issue number

#478

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • Add a new news fragment into the CHANGES folder
    • name it <issue_id>.<type> (e.g. 588.bugfix)
    • if you don't have an issue_id change it to the pr id after creating the PR
    • ensure type is one of the following:
      • .feature: Signifying a new feature.
      • .bugfix: Signifying a bug fix.
      • .doc: Signifying a documentation improvement.
      • .removal: Signifying a deprecation or removal of public API.
      • .misc: A ticket has been closed, but it is not of interest to users.
    • Make sure to use full sentences with correct case and punctuation, for example: Fix issue with non-ascii contents in doctest text files.

@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Merging #479 (e065c00) into master (99c1809) will not change coverage.
The diff coverage is 75.00%.

@@           Coverage Diff           @@
##           master     #479   +/-   ##
=======================================
  Coverage   85.57%   85.57%           
=======================================
  Files          31       31           
  Lines        1837     1837           
  Branches      266      266           
=======================================
  Hits         1572     1572           
  Misses        188      188           
  Partials       77       77           
Flag Coverage Δ
unit 85.24% <75.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
aiohttp_debugtoolbar/panels/traceback.py 88.23% <0.00%> (ø)
aiohttp_debugtoolbar/panels/middlewares.py 80.00% <100.00%> (ø)
aiohttp_debugtoolbar/tbtools/tbtools.py 59.64% <100.00%> (ø)
aiohttp_debugtoolbar/views.py 81.65% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Dreamsorcerer Dreamsorcerer changed the title Hacky fix for broken static urls with yarl>1.8.2 Fix static urls with yarl 1.9+ Sep 13, 2023
@Dreamsorcerer Dreamsorcerer merged commit e28307a into aio-libs:master Sep 13, 2023
19 of 20 checks passed
@Dreamsorcerer
Copy link
Member

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

yarl update broke static file urls in templates
2 participants