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

Skip to content

Upgrade Starlette to version 0.16.0#3681

Closed
jerluc wants to merge 1 commit into
fastapi:masterfrom
StationA:starlette-0.16
Closed

Upgrade Starlette to version 0.16.0#3681
jerluc wants to merge 1 commit into
fastapi:masterfrom
StationA:starlette-0.16

Conversation

@jerluc

@jerluc jerluc commented Aug 10, 2021

Copy link
Copy Markdown

Overview

This small change upgrades FastAPI's internal dependency on Startlette to version 0.16.0, which incorporates several important bugfixes (see release notes for 0.15.0 and 0.16.0) that would be relevant for FastAPI users.

Testing done

  • Run unit tests per the contribution docs
  • Run a few examples from the FastAPI docs locally to ensure some "real-world" examples still work as expected

Other notes

  • During the upgrade, there were only a couple of unit tests that failed due to upstream breakage relating to websocket connections, which were pretty straightforward to resolve
  • It's worth noting that Starlette now requires AnyIO as of version 0.15.0; I don't claim to fully understand 100% of the reasons behind this switch, but from the commits I've read in Starlette, it seems to provide upstream support for different async runtimes (e.g. Trio) along with other nice async I/O primitives such as task groups

Apologies in advance if version upgrades aren't a typical part of outside contribution, but I figured this would be one way to get my feet wet!

@Kludex

Kludex commented Aug 11, 2021

Copy link
Copy Markdown
Member

Duplicated #3594

@brnosouza

Copy link
Copy Markdown

any updates on this?

@jerluc

jerluc commented Nov 12, 2021

Copy link
Copy Markdown
Author

It looks like this is has been done in #4016 and is available as of FastAPI v0.70.0

@jerluc jerluc closed this Nov 12, 2021
@jerluc jerluc deleted the starlette-0.16 branch November 12, 2021 20:27
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