-
Notifications
You must be signed in to change notification settings - Fork 59
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 framework instrumentation #40
Comments
Imo they should definitely be in a separate repo each. They should be able to get their own independent versioning. Noone uses all these frameworks at the same time, so they also shouldn't have to pull in code for it. |
Integration with Symfony: https://github.com/jcchavezs/zipkin-instrumentation-symfony |
For laravel, there are this libraries so far: |
Is this something to still look into? Laravel seems supported and Silex is quite dead. There are some modern frameworks around (Symlex, Lumen, Spiral) Lumen seems to have https://github.com/shlj/php-zipkin-lumen |
There is a PR for laravel. It would be really cool if we could add a test
to make sure laravel works fine. Would you be up to do that?
|
We need to provide framework integrations for zipkin. Priority frameworks are:
Ping @felixfbecker @beberlei
The text was updated successfully, but these errors were encountered: