microsoft / ApplicationInsights-PHP

Azure Application Insights SDK for PHP
https://azure.microsoft.com/services/application-insights/
Other
108 stars 92 forks source link

Add default operation id on client creation #51

Closed jondmcelroy closed 6 years ago

jondmcelroy commented 6 years ago

When I started using this library I didn't know that you needed to set the operation id in order to link calls. I added it by default since I'm not sure anyone would want this without it. I also updated the documentation to be more clear about the other possible options.

jjindrich commented 6 years ago

I checked your PR, but I don't see how you setup Operation_name. I documented on my GitHub https://github.com/jjindrich/jjazure-web-php/blob/master/jjwebphpai/src/index.php. But I'm not sure what is exactly required to have working all reports.

jondmcelroy commented 6 years ago

@jjindrich Yes, this PR is only looking at the Operation ID, not the name.