-
Notifications
You must be signed in to change notification settings - Fork 29
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
BackgroundTasks don't run #212
Comments
Thanks! I'll try to provide the fixes tomorrow! |
Found the reason. Will publish the fix within 1-3 days. |
Thanks for the updates! |
@jfeaver please, check version 4.2.4 and see if it fixes your problem. I have also added a unit test for this behavior to make sure we never have such issues again. |
Great work. Thanks! Background tasks are working as expected for me. |
@jfeaver thank you for publishing these bug reports! They really do help a lot. If you have any extra feedback, any questions, or requests for Cadwyn -- feel free to ping me here or in our discord. If you need any help integrating Cadwyn into your organization or consulting regarding API design/versioning -- feel free to contact me as well. |
Describe the bug
BackgroundTasks
don't run.To Reproduce
Use this Cadwyn app:
POST
to the/send-notification/{email}
route.Expected behavior
A log file should be created with the provided email param logged.
This vanilla FastAPI App has the expected behaviour:
Operating system
MacOS 14.3.1 (23D60)
Additional context
cadwyn v4.2.3
fastapi v0.115.0
The text was updated successfully, but these errors were encountered: