-
-
Notifications
You must be signed in to change notification settings - Fork 189
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
Elasticsearch support #672
Comments
Hey @nklmilojevic ! Cool thing you want to use Sentry more. Currently there is no ElasticSearch support in PHP, you are right. But as always: PRs are very welcome! If you want to give it a go, I would look at how we do support for databases and copy this and change it. |
@nklmilojevic Are you using https://github.com/elastic/elasticsearch-php or something else? |
Hi @cleptric! We use https://github.com/cviebrock/laravel-elasticsearch which essentially wraps the official client into Laravel. |
is kind of a red flag 🙁 |
It is - we planned to fork it and to maintain it as MailerLite company (of course, opensourced), and that will probably happen in the next few days. |
The nice thing is that you can already enable tracing of the Elastic client. $elasticClient = Elastic\Elasticsearch\ClientBuilder::create()
->setHttpClient($someHttpClient)
->build(); The Elastic $stack = GuzzleHttp\HandlerStack::create();
$stack->push(Sentry\Tracing\GuzzleTracingMiddleware::trace());
$httpClient = new GuzzleHttp\Client([
'handler' => $stack,
]);
$elasticClient = Elastic\Elasticsearch\ClientBuilder::create()
->setHttpClient($someHttpClient)
->build(); This |
Hi!
We are in the process of switching from Datadog to Sentry fully (paid) and one thing I noticed is that there is no Elasticsearch traces. Is that something that is planned to support?
Thanks!
The text was updated successfully, but these errors were encountered: