633WHU comments

Results 30 comments of


                                            633WHU

307 Temporary Redirect

#781

Fix 307 Temporary Redirect when use streamable_http

#732

Fix 307 Temporary Redirect when use streamable_http

when we set the default /mcp, the log will be, in some env will ``` INFO: Started server process [55547] INFO: Waiting for application startup. INFO: Application startup complete. INFO:...

Fix 307 Temporary Redirect when use streamable_http

@simonw @sheffler @comfuture @shimizukawa

Fix 307 Temporary Redirect when use streamable_http

@ihrpr

Fix 307 Temporary Redirect when use streamable_http

> Thanks for identifying and reporting this redirect issue! However, I don't think changing the default streamable_http_path is the right approach here. > > The core issue: This change creates...

Fix 307 Temporary Redirect when use streamable_http

> Why are we even mounting applications? We can have a `Router` and set `redirect_slashes` to `False`. I have modified it according to your suggestion. Could you please review it?...

Fix 307 Temporary Redirect when use streamable_http

Updated the latest code, please help review @Kludex @vectorstain @mroch @ihrpr

Fix 307 Temporary Redirect when use streamable_http

> You marked my comments as resolved, but you actually ignore them. Restored, please check one by one

The fixed value for feel_zeroes in get_binned_data may lead to deviation in some case.

> Wow, I managed to mess up a bit with this issue by adding many different references in my local commits! Sorry!! (new to contributing here) > > I added...