Modify line 136 of watchdog.module to provide a clickable IP address.

The following provides a way to resolve a logged IP address easily.

$output .= ' <tr><th>'. t('Hostname') .'</th><td><a href="http://www.whois.sc/' . $watchdog->hostname . '" target="_blank">' . $watchdog->hostname. '</a></td></tr>';

Something nice and easy to add in. Good to track strange people logged in to Drupal and doing stuff they maybe shouldn't be.

CommentFileSizeAuthor
#4 jsomers_16221_whois.patch655 bytesj.somers

Comments

sneex’s picture

sneex commented

I vote +1 on this (I already added it myself, but) ... I vote for same window and not a new one -- maybe make it configurable?

$output .= '

'. t('WhoIS') ." Display!

";

forngren’s picture

forngren commented
Version: 4.5.2 » x.y.z

I can't see why Drupal needs this feature, but on the other hand; why not?

LAsan’s picture

LAsan commented
Version: x.y.z » 7.x-dev

Moving to cvs

j.somers’s picture

j.somers commented
StatusFileSize
new jsomers_16221_whois.patch655 bytes

I also found this useful and always did it myself.

Attached is a patch that uses http://whois.domaintools.com.

j.somers’s picture

j.somers commented
Status: Active » Needs review

Forgot to change this.

On one hand I could doubt if functionality like this is really needed in core, but on the other hand there doesn't seem to be a way to add this using a hook.

chx’s picture

chx commented

Anyone asked domaintools whether they want a barrage like this?

budda’s picture

budda
Primary language English
Location Cheshire
commented

As its not an automated process, so there would not be a barrage of requests, unless the site admin went through clicking each IP address in the log one at a time.

Probably courteous to let them know Drupal is considering integrating their service though.

dries’s picture

dries commented

I agree that we should check their terms of service and maybe even their privacy policy.

Status: Needs review » Needs work

The last submitted patch failed testing.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Version: 7.x-dev » 8.x-dev

Isn't this a duplicate of #22494: Add IP lookup capabilities to the watchdog module?

devin carlson’s picture

devin carlson commented
Status: Needs work » Closed (duplicate)

Yes, this is a duplicate. I'm marking this issue as the duplicate because it seems like more work seems to have been done in the other issue.

Duplicate of #22494: Add IP lookup capabilities to the watchdog module