It would be useful to have the host as well as the full request URI, so that if multiple sites feed to the same logger, it's easy to separate them out.

In Drupal 6, we used to use the gelf module, which added "Server_host" in #1247572: add Server Name for identifying separate environments

CommentFileSizeAuthor
#2 host-3088690-2.patch561 bytesmalcomio
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

malcomio created an issue. See original summary.

malcomio’s picture

malcomio CreditAttribution: malcomio at Capgemini commented
Status: Active » Needs review
FileSize
host-3088690-2.patch561 bytes
jonathan_hunt’s picture

jonathan_hunt CreditAttribution: jonathan_hunt at Catalyst IT commented

Patch working for me ok, thanks.

RoSk0’s picture

RoSk0
he/him
Primary language Russian
Location Wellington
CreditAttribution: RoSk0 at Catalyst IT commented
Status: Needs review » Needs work

While host name might be a useful information to have, I don't see how it is related to "RequestUriProcessor". Feels like it should be a separate processor.

lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
CreditAttribution: lussoluca commented
Status: Needs work » Fixed

Committed and pushed to 2.x-dev. Thanks!

I add this as a new Processor.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.