Semafind / semadb

No fuss multi-index hybrid vector database / search engine
https://semadb.com/
Apache License 2.0
9 stars 2 forks source link

Revert /ping back to App middleware #7

Closed nuric closed 9 months ago

nuric commented 9 months ago

This reverses #6

The ping endpoint is part of the v1 api which is documented as requiring user access. This means who is pinging the server is needed.

We are going to leave the /ping endpoint within app headers for now because we don't want potentially unauthenticated requests coming in. It may be tricky to remember which endpoints require user ID and plan with those that do not. This way, we know every request is treated equally and that makes management less error prone.