slgobinath / uget-chrome-wrapper

Moved to https://github.com/ugetdm/uget-integrator and https://github.com/ugetdm/uget-extension
GNU General Public License v3.0
75 stars 14 forks source link

Why some files are ignored while some are downloaded #7

Closed doylefermi closed 7 years ago

doylefermi commented 7 years ago

Screencast

Using the latest version 1.1.2 Some files aren't captured while some are. Is there something to be configured?

slgobinath commented 7 years ago

uGet Integration extension captures the downloads with HTTP 200 status if the file size is greater than 300 kb. However, I also noticed that it does not capture the downloads from SaveFrom.net (Earlier it captured them but now I think the SaveFrom.net changed it's own behavior). I am working on it and I will release the fix as soon as possible. thanks for reporting this issue.

Note: If the SaveFrom.net link is opened in a new tab, uget-chrome-wrapper captures them.

slgobinath commented 7 years ago

From uGet Integration extension v1.1.3, an option to Download with uGet is added to the right click context menu of links. I hope it will help you until I resolve the exact problem.

This is what you have to do:

  1. Right click on the link
  2. Select Download with uGet
doylefermi commented 7 years ago

Yes that's a good workaround.

The response from SaveFrom.net is a HTTP 302 and its a redirection. I guess HTTP 200 alone shouldn't be considered.

slgobinath commented 7 years ago

Fixed with https://github.com/slgobinath/uget-chrome-wrapper/commit/b9d9a19bed38694b6b4088dbddfdc754b5a4667b and the fix is added in v1.1.4.

doylefermi commented 7 years ago

Yes that's a perfect fix. Thanks.

But now Insert cannot stop capturing the downloads.

Also when downloading from savefrom.net, single click captures the download with filename videoplayback while Rightclick->OpenLinkInNewTab captures the original filename.

slgobinath commented 7 years ago

Yes I've noticed them. Once I find a fix, I will release a new version :-)

slgobinath commented 7 years ago

@doylefermi Please check the new release with uget-chrome-wrapper v1.3.3 and uGet Integration extension v1.1.6.

This version has the following:

doylefermi commented 7 years ago

Thanks. It works now.