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.

Sign up for GitHub

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

Jump to bottom

Add duplicate /vectors route to remove 307 redirects, fixes #34 #43

Merged

antas-marcin merged 1 commit into main from trengrj-307

Jan 16, 2023

Member

trengrj commented Jan 16, 2023 •

edited

Loading

Currently, Weaviate is configured to POST to /vectors however the module listens on /vectors/ causing many 307 HTTP redirects:

INFO:     127.0.0.1:59650 - "POST /vectors HTTP/1.1" 307 Temporary Redirect
INFO:     127.0.0.1:59651 - "POST /vectors HTTP/1.1" 307 Temporary Redirect
INFO:     127.0.0.1:59662 - "POST /vectors/ HTTP/1.1" 200 OK
INFO:     127.0.0.1:59663 - "POST /vectors/ HTTP/1.1" 200 OK

This PR adds a duplicate route to remove the 307. It is non breaking as we leave the original route alone (though we should deprecate it).


          Add duplicate /vectors route to remove 307 redirects

07b0395

antas-marcin merged commit b62e425 into main

antas-marcin deleted the trengrj-307 branch

January 16, 2023 09:15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet