chatgpt-google-extension icon indicating copy to clipboard operation
chatgpt-google-extension copied to clipboard

Non-flatpack Linux: This add-on is not available on your platform.

Open chapmanjacobd opened this issue 2 years ago • 6 comments

On OpenSuSe Firefox I get the message "This add-on is not available on your platform." in Mozilla Addons.

Also, "Access your data for all websites" seems pretty aggressive. Think you could make it only use www.google.com and *.openai.com ?

chapmanjacobd avatar Dec 05 '22 17:12 chapmanjacobd

Can you install other addons in the Mozilla Addons site?

wong2 avatar Dec 05 '22 17:12 wong2

Yes. Also I checked my other computer running Fedora and I get the same message

chapmanjacobd avatar Dec 05 '22 17:12 chapmanjacobd

I can use it in Flatpak version of Firefox on Linux without a problem. Just so you know.

omergoktas avatar Dec 05 '22 19:12 omergoktas

Also, "Access your data for all websites" seems pretty aggressive. Think you could make it only use www.google.com and *.openai.com ?

Google has about 200 country-specific search domains. It's not possible to limit the manifest matches key further without specifying all 200 of them as a big list. There's an additional restriction applied in include_globs so that the script only runs on "https://www.google./", but the addon store UI can't tell you that.

marauder37 avatar Dec 05 '22 21:12 marauder37

Thanks @marauder37 that's right

wong2 avatar Dec 06 '22 02:12 wong2

@chapmanjacobd Have you tried to manually install the extension by following README?

wong2 avatar Dec 06 '22 13:12 wong2

It's working now. I wonder if Mozilla flagged it temporarily ? I don't know

chapmanjacobd avatar Dec 07 '22 04:12 chapmanjacobd

It's not possible to limit the manifest matches key further without specifying all 200 of them as a big list.

Still, it's probably worth it to explicitly list them, so people have ease of mind. It may be true that there is restriction applied in include_globs but I won't be notified if that changes. For me personally, I won't be using this given the aggressive access requirement, although I really really want to. @wong2 If you want your user base to grow exponentially I highly recommend treating the access list seriously. I see huge potential in this add-on. Feel free to ignore me if it's just intended to be a hobby project though.

nkstnkst avatar Dec 08 '22 19:12 nkstnkst

@nkstnkst Thanks, I'll figure this out.

wong2 avatar Dec 09 '22 02:12 wong2

Hi there, I see the issue is closed, but I'd like to give my 2 cents too on this topic @wong2

I would also like to give support to the idea of "Principle of Least Privilege" if that's possible. If the problem is explicitly listing 200 domains, this looks quite feasible as it is a quite static content, so not many updates will be required in the future. If the problem is another thing (I'm not sure at all), it would be very nice of try addressing it with a more fine grained permission control.

When a user (including myself) faces this prompt, they think twice if go ahead or not:

image

The fact that the project is Open Source gives an extra point of confidence, but still not everyone will have the time or knowledge to validate it.

I hope this feedback can help in any sense :)

Cheers!

Relates to: https://github.com/wong2/chat-gpt-google-extension/issues/35#issuecomment-1339408344, https://github.com/wong2/chat-gpt-google-extension/issues/35

gerardbosch avatar Dec 10 '22 15:12 gerardbosch

Ooops! I've just seen this was somehow addressed yesterday here https://github.com/wong2/chat-gpt-google-extension/commit/ce17eeb248e058361abca2d9fd972454ce62731d

But I've just tried to install the extension from Chrome Web Store and the "Read and change all your data on all websites" was requested.

gerardbosch avatar Dec 10 '22 15:12 gerardbosch

@gerardbosch yeah resolved in v1.3.0 still pending for review in Web store

wong2 avatar Dec 10 '22 15:12 wong2

@gerardbosch yeah resolved in v1.3.0 still pending for review in Web store

Awesome! Thanks so much for answering and so fast :smiley:

gerardbosch avatar Dec 10 '22 15:12 gerardbosch