search

microsoft / ApplicationInsights-WordPress

Main development repository for Application Insights WordPress plugin.
MIT License
45 stars 42 forks source link

404 Tracking Settings and fixed opcache issue #23

Closed dulfe closed 5 years ago

dulfe commented 6 years ago

This PR addresses an issue where php complains than the Settings.php file does not exists when Zend Opcode is in use. I tracked down the issue to the way the "autoloader" was been referenced in ApplicationInsightsPlugin.php.

It also adds the option to opt-out of 404 tracking and lets the user mark some files as "non-trackable 404s" which is useful for files that some browsers assume exists but they are optional, like favicon.ico, robots.txt, etc.

c-fitzmaurice commented 5 years ago

I'm not sure about this rest of this PR, but I can confirm that adding the code below to ApplicationInsightsPlugin.php helped. Without it, I was unable to use the wp-cli from the Kudo console.

require_once __DIR__ . '/vendor/autoload.php';

Merging this would be much appreciated, thank you.