✨ Add support for setting transaction name to path in FastAPI #1349
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
✨ Add support for setting transaction name to
path
in FastAPIThis is related to #829 and #1165
This would work with FastAPI
0.74.1
that I just released 5 minutes ago.Most of the functionality is already tested by the Starlette tests. I added a FastAPI test only to test the new
transaction_style="url"
. But I'm not sure if that's desired. If it is, FastAPI would have to be added to the testing dependencies here, but I'm not even sure where, I suspect somewhere in the Tox configs, but I don't really know much about Tox 😬 , so I'll wait for feedback first.