Discussions on scripts by B10m | Userscripts.org

Archived Comments, replied by B10m

B10m — Fri, 09 Mar 2007 20:19:44 +0000

Thanks for the feedback nitro322. Let me explain why I don't think your examples should be updated:

link: http://www.evilsite.net/
text: www.goodsite.net

Technically, "www.goodsite.net" is not an URL. I don't trust that to begin with ;-). We could scan for text that begins with 'www', but the next thing would be to include 'secure', 'images', etc. What about this?

link: http://evilsite.com/
text: goodsite.com

As you can see, it's getting tricky.

link: http://www.evilsite.net/
text: goodpage.html

See above. This URL is not really trying to trick you that much. Following this example (and taking it to extremes) this would be incorrect too:

link: http://www.evilsite.com/contact.html
text: faq

link: http://www.goodsite.net/evilpage
text: http://www.goodsite.net/

The same thing as above applies here. This isn't really a fishing attempt. We're on the same domain, so the least we can do is trust the server. The "evilpage" shouldn't be on the "goodsite.net" to begin with ;-)

Archived Comments, replied by nitro322

nitro322 — Sun, 25 Feb 2007 07:28:26 +0000

By the way, the link/text pairs in my example should have all been plain text. The userscript.org website translated them into links. I'd recommend not clicking on any of them as I really don't know where they lead.

Archived Comments, replied by nitro322

nitro322 — Sun, 25 Feb 2007 07:26:32 +0000

B10m, I've been using this script for a while now. Very cool idea; thanks for making it available.

I've recently been testing it, though, and I noticed a few cases where it will fail. For example, take the following link/text pairs:

link: http://www.evilsite.net/
text: www.goodsite.net

link: http://www.evilsite.net/
text: goodpage.html

link: http://www.goodsite.net/evilpage
text: http://www.goodsite.net/

In all three cases, your script will fail to report the discrepancy. I tried playing around with the source a bit to see if I could make this work, but I'm just not that good with Javascript.

Any chance you could look into this and try to make your script a bit more robust?

Archived Comments, replied by nick 4

nick 4 — Thu, 30 Nov 2006 03:39:48 +0000

It's now embedded directly into the site, so no greasemonkey required. Thanks for your permission!

http://30boxes.com/blog/index.php/2006/11/29/go...

Archived Comments, replied by B10m

B10m — Thu, 30 Nov 2006 00:35:45 +0000

Sure thing, implement it so I can delete this script ;-)

Archived Comments, replied by nick 4

nick 4 — Wed, 29 Nov 2006 23:27:29 +0000

This is awesome. So good, in fact, I'd like to implement it as a normal part of the site. Is that okay with you B10m? We'll give you credit on our blog for the great idea and easy code, and send you a tshirt whenever we get around to making some...

Archived Comments, replied by Narendra

Narendra — Wed, 29 Nov 2006 20:11:45 +0000

that is seriously excellent! I will blog it on 30B

Archived Comments, replied by BOK

BOK — Wed, 29 Nov 2006 18:31:02 +0000

This is like an extra pinch of salt on 30 Boxes! Thanks for the script, b10m.

Archived Comments, replied by Paul Irish

Paul Irish — Sun, 01 Oct 2006 16:00:47 +0000

I've created an expansion of this script for ppt, xls, and doc. Along with some other enhancements. http://userscripts.org/scripts/show/5646

Archived Comments, replied by Paul Irish

Paul Irish — Fri, 15 Sep 2006 16:08:52 +0000

What's the regex adjustment so this applies to files that end in caps: .PDF ?
Also I might have to augment this for .doc, .xls, and .ppt.
This is great.

Archived Comments, replied by Paul Irish

Paul Irish — Wed, 13 Sep 2006 16:04:32 +0000

I just had this idea yesterday and was pleased to see you already implemented it. Nicely done.

Archived Comments, replied by mar 1

mar 1 — Mon, 15 May 2006 00:12:43 +0000

This is certainly one of the best Greasemonkey scripts that I've ever used. Personaly, I want to know what is hiding under the surface. So, this script has made it much easier for me to detect suspicious Web pages. If you are using the "Redirect Remover" extension (which I am doing), it will be even better to have this script installed since "Gone Fishing" will act as an early-warning-system when you click on a suspicious link. I recommend them both! If you want the extension to work, you have to configure it in a certain way. For best result, go to the "Highlighting" tab and untick the first checkbox (...containing a removed redirect). For the uncleaned section, just use yellow as background. Sorry for the looong comment! B10m you rock!! :)

Archived Comments, replied by Wulf

Wulf — Fri, 07 Apr 2006 19:30:29 +0000

Great script, I just wish the images were a bit bigger... 15px seems to be optimal for most inline images.

Archived Comments, replied by B10m

B10m — Fri, 03 Feb 2006 14:40:21 +0000

Updated, thanks a bunch.

Archived Comments, replied by Bret Mogilefsky

Bret Mogilefsky — Fri, 03 Feb 2006 12:37:24 +0000

Works great, and a great idea. Thanks!

One little nit... The script was turning up false positives over differences in case, which I think is unnecessary. That situation crops up more often than you might think; about 2/3 of the warnings I was seeing were for that. I put ".toLowerCase()" on each of the operands to the main string compare to stop that.

Archived Comments, replied by B10m

B10m — Wed, 25 Jan 2006 18:27:16 +0000

yes -late reply- the changes have been implemented a long while ago. The images are now _in_ the script, as wisely pointed out by "ankut".

Archived Comments, replied by Wulf

Wulf — Sat, 31 Dec 2005 00:47:46 +0000

Sounds like a great script :)

The update dates here always seem to be the current date for me... Has this been updated to include the images [i]in[/i] the script yet?

Archived Comments, replied by Kevin Richards

Kevin Richards — Sat, 10 Dec 2005 07:12:33 +0000

This is a cool script. What might be a good idea though, particularly for e-mail links is to append the image to the front of the link rather than the rear. This prevents breaking up text in cases where they tack on 's (or other things) to the end of the e-mail address but don't make it a part of the link.

Archived Comments, replied by B10m

B10m — Mon, 10 Oct 2005 03:25:18 +0000

Updated, thanks for informing me about this!

Archived Comments, replied by ankut

ankut — Sun, 09 Oct 2005 19:19:55 +0000

Also, don't link to images from your script!! Inline the image data into the script itself by using the URI Kitchen utility

Archived Comments, replied by B10m

B10m — Sun, 09 Oct 2005 08:46:10 +0000

Many other icons could be added (.xls, .doc, etc.) yet mailto and PDF links were my biggest annoyance, so there you have it ;)

However, feel free to update this script, give it a somewhat better name and release it. Just let me know.

Archived Comments, replied by Jesse Andrews

Jesse Andrews — Wed, 17 Nov 2004 01:05:56 +0000

The following is an archive of comments made before threaded discussions was implemented (November 16th, 2008)

Archived Comments, replied by Jesse Andrews

Jesse Andrews — Wed, 17 Nov 2004 01:05:54 +0000

The following is an archive of comments made before threaded discussions was implemented (November 16th, 2008)

Archived Comments, replied by Jesse Andrews

Jesse Andrews — Wed, 17 Nov 2004 01:04:57 +0000

The following is an archive of comments made before threaded discussions was implemented (November 16th, 2008)