I have a page with a link that points to a 700 MB file:
page: http://amedee.be/ubuntu-acer-aspire-one-easy-way
link to: http://ftp.belnet.be/mirror/ubuntu.com/releases/8.10/ubuntu-8.10-mid-lpi...

Urlicon tries to fetch the entire 700MB file in an attempt to display an icon.
This results in an unreasonable high amount of traffic.
Before I found that Urlicon is causing the traffic, I blacklisted the IP address of ftp.belnet.be in my firewall.

Now that I have found the cause, I will change Filter URLs in admin/settings/filters from Find all external URLs and append the according favicon (if available) to Find all external URLs and append an external link icon.
I am even considering uninstalling the module. Too bad, because it looks nice. But the unexpected traffic was just too much.

CommentFileSizeAuthor
#5 patch_commit_d15ab5a31882.patch3.33 KByvmarques
#2 urlicon.module.test9.95 KBfunkytraffic

Comments

netsensei’s picture

netsensei commented

Subscribing

funkytraffic’s picture

funkytraffic commented
StatusFileSize
new urlicon.module.test9.95 KB

I think URLIcon is in general making a lot of traffic, which is a mayor bug, since it is searching for new icons allways, even if the site does not have any.

For this reason I wrote another option to the module, not to search for new icons, only use the saved ones.

dawehner’s picture

dawehner
Primary language German
commented

Please provide a patch, this is the only way to add this to the module. Thanks!

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented

Well, we could switch from a get to a head request.

yvmarques’s picture

yvmarques commented
StatusFileSize
new patch_commit_d15ab5a31882.patch3.33 KB

Hello,

I created a patch that check by HEAD request before download the file and see if it's a html file.

Regards,

-- Yvan

yvmarques’s picture

yvmarques commented
Status: Active » Needs review
funkytraffic’s picture

funkytraffic commented

I would rather suggest to rewrite the module to search for new icons only on saving a node and using the filter to search only for saved icons.

This would save traffic.

Second point would be to save all icons in one file, but this is probably to much work, though would result in a gain of time. The themed link then only would give a certain position on a css background styling for the icon, having one icon file, like:

http://www.nachrichten.de/images/favicons.png

This would rather be an idea for a 7.x release since then the image module is in core.