hmm, oh, this was probably working before because FastAPI had a `span_modifier` defined, so it didn't require Starlette to be patched in order to work.    Now with this change... are we now *requiring* Starlette to be patched in order to use FastAPI?    Should we replicate some of the patching in Starlette into FastAPI?