Skip to content
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.

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

Add Typesense health check #69

Closed
wants to merge 3 commits into from
Closed

Conversation

dyanakiev
Copy link

Its based on the MeiliSearch check.
Basically, we are checking the health url http://127.0.0.1:8108/health of Typesense and it should contain {"ok": true} if healthy, doesn't have any other type of responses.

Typesense docs: https://typesense.org/docs/guide/install-typesense.html#%F0%9F%86%97-health-check

Its based on the MeiliSearch check. 
Basically, we are checking the health url http://127.0.0.1:8108/health of Typesense and it should contain `{"ok": true}` if healthy, doesn't have any other type of responses.

Typesense docs: https://typesense.org/docs/guide/install-typesense.html#%F0%9F%86%97-health-check
@freekmurze
Copy link
Member

Which PHP package adds that TypesenseCheck?

@dyanakiev
Copy link
Author

Which PHP package adds that TypesenseCheck?

I added the 'TypesenseCheck' here in this repo you can see my three commits in the pull request.

@freekmurze
Copy link
Member

Ah, I missed that 🙂

Because I don't want to maintain the code myself, I would prefer you put it in a separate package of your own.

Take a look at this check as an example: https://spatie.be/docs/laravel-health/v1/available-checks/ssl-certificate-expiration

@freekmurze
Copy link
Member

Create a new PR with docs after you've created your own package.

@freekmurze freekmurze closed this Feb 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants