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

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

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

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

Moving to cvs

j.somers’s picture

StatusFileSize
new655 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

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

Anyone asked domaintools whether they want a barrage like this?

budda’s picture

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

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

Version: 7.x-dev » 8.x-dev
devin carlson’s picture

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