-
Notifications
You must be signed in to change notification settings - Fork 3
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
Switch deprecated startup events to lifespan events #152
Comments
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days.
|
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days.
|
Hi @alyssadai, Can i take this issue? 😋 |
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days.
|
The
@app.on_event("startup")
syntax seems to be deprecated: https://fastapi.tiangolo.com/advanced/events/#alternative-events-deprecated and replaced by https://fastapi.tiangolo.com/advanced/events/#lifespanWe should look into updating the startup checks for defined credentials and allowed origins to follow this new syntax.
Originally posted by @surchs in #151 (comment)
The text was updated successfully, but these errors were encountered: