search

Pierre-Lannoy / wp-decalog

Capture and log events, metrics and traces on your site. Make WordPress observable - finally!
https://perfops.one/
GNU General Public License v3.0
64 stars 8 forks source link

Add sitedomain to event logging #16

Closed JanThiel closed 3 years ago

JanThiel commented 3 years ago

Is your feature request related to a problem? Please describe. Currently there is the siteid and the sitename within the meta of an event log entry. The name may not be easily matchable to a single site in a multisite environment. The id again requires an additional lookup step to identify the origin site. Having the domain would allow a direct access to the origin site (/wp-admin for example). In addition to that the domain is more stable than the name of a site and thus better suited as a readable identifier within tools like Datadog. Currently one would have to know the ID to create stable filters / queries. The domain would be better.

Describe the solution you'd like Add a new field sitedomain containing the sites domain.

Describe alternatives you've considered None. Small change, big effect

Pierre-Lannoy commented 3 years ago

Hi Jan! You're definitely right. That's a good thing to have. Let me review your PR....

Pierre-Lannoy commented 3 years ago

Many thanks, Jan, for your PR. It will be in DecaLog 3.2.0...

JanThiel commented 3 years ago

Thank you very much @Pierre-Lannoy for this amazing Toolset :-) Glad to contribute.