text-embeddings-inference icon indicating copy to clipboard operation
text-embeddings-inference copied to clipboard

Published 20 hours ago • verified publisher icon huggingface

Better HTTP Status Code for Empty Requests

Open mbbyn opened this issue 1 year ago • 5 comments

System Info

(I don't think it's necessary)

Information

  • [X] Docker
  • [ ] The CLI directly

Tasks

  • [X] An officially supported command
  • [ ] My own modifications

Reproduction

image

Expected behavior

HTTP status 400 is more appropriate IMHO. The 413 status code might lead to misinterpretation. For example, when integrated with Dify, dify translates the 413 to Payload too large, but there is not payload to begin with.

mbbyn avatar Aug 01 '24 09:08 mbbyn