Thanks to visit codestin.com
Credit goes to github.com

Skip to content

The referenced example code is the one for the TrustedHostMiddleware#1146

Closed
JensGe wants to merge 1 commit into
fastapi:masterfrom
JensGe:patch-1
Closed

The referenced example code is the one for the TrustedHostMiddleware#1146
JensGe wants to merge 1 commit into
fastapi:masterfrom
JensGe:patch-1

Conversation

@JensGe

@JensGe JensGe commented Mar 20, 2020

Copy link
Copy Markdown

advanced_middleware/tutorial002.py > advanced_middleware/tutorial003.py

advanced_middleware/tutorial002.py > advanced_middleware/tutorial003.py
@codecov

codecov Bot commented Mar 20, 2020

Copy link
Copy Markdown

Codecov Report

Merging #1146 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1146   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          331       331           
  Lines         8085      8085           
=========================================
  Hits          8085      8085           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update faf88ce...1589053. Read the comment docs.

@arimbr

arimbr commented Mar 20, 2020

Copy link
Copy Markdown
Contributor

Feels like we were other two to open a PR:
#1068
#1138

PR #1138 also changes hl_lines to match the highlighted code lines for the new block.

@tiangolo

Copy link
Copy Markdown
Member

Thanks for your contribution @JensGe ! 🍰

This was fixed in #1138 by @arimbr 🤓

@tiangolo tiangolo closed this Mar 27, 2020
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.

3 participants