Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow whitelisting some websites #2

Closed
martin-braun opened this issue Sep 13, 2021 · 11 comments
Closed

Allow whitelisting some websites #2

martin-braun opened this issue Sep 13, 2021 · 11 comments
Labels
enhancement New feature or request

Comments

@martin-braun
Copy link

martin-braun commented Sep 13, 2021

There are cases where Undisposition is not useful. For instance when downloading a certificate from Apple's Certificate, Identifiers & Profiles. This is just one case. Some websites fail to provide the right headers so this extension can't work out what is binary or not, thus attempting to show binary data as text or naming the filename wrong.

In case of Apples certificate website, it names the *.cer file "downloadCertificateContent.action", which is wrong.

Being able to white list critical websites would give more control to the user.

@nihaals
Copy link

nihaals commented Sep 16, 2021

Another example of a site that stops working is Google Drive

@Racle
Copy link
Owner

Racle commented Sep 17, 2021

I'll look into this andI do not promise any time frame for this.

And If/when I get this done, this feature is coming as right click menu option as I do want to keep extension icon as fast toggle.

Currently extension is checking URL and getting extension from that (very simple solution).
If that extension finds commonly used extensions, it has checks for those.

@Racle Racle added the enhancement New feature or request label Sep 17, 2021
@martin-braun
Copy link
Author

@Racle Thanks, yes, you don't have to provide any ETA, I'm thankful for this extension nonetheless. Often I can simply press CMD+S to store the file anyway. In case of wrong filenames, I can also just rename the file to what it should supposed to be. However, a few times I fell in this pitfall, because I forgot.

I agree that a context menu implementation is enough. A dev who discovers problems will attempt to try to right click to find a white list feature anyway.

Thanks for your fork.

@nihaals
Copy link

nihaals commented Sep 22, 2021

Having a default blacklist could be helpful in instances where downloading just doesn't work. For example, with Google Drive, you get an error about third party cookies not being enabled and there is no other way to download the file. Since the error message is wrong, it will likely lead to people looking into different solutions without thinking this was the cause.

@orschiro
Copy link

Same issue for me.

It sadly breaks Google Drive download.

Doesn't happen if I disable the extension.

image

@Racle
Copy link
Owner

Racle commented Oct 23, 2022

@martin-braun @nihaals @orschiro
Just uploaded version 0.0.5 with (very simple) blacklist to Firefox and Chrome web stores.
You can find Settings by right clicking extension icon.

This should be approved soon.

Chrome version also has manifest v3 support, so that should work even after that is forced.

@Racle Racle closed this as completed Oct 23, 2022
@orschiro
Copy link

orschiro commented Oct 23, 2022 via email

@nihaals
Copy link

nihaals commented Oct 28, 2022

I'm not sure if Manifest v3 supports it, but it would be nice to be able to use the browser's built-in settings button too (options_page/options_ui) to stay consistent with other extensions.

@nihaals
Copy link

nihaals commented Oct 28, 2022

Created as #4

@martin-braun
Copy link
Author

@Racle Should we maintain a proper default black list? I see apple.com and google.com made it, but dropbox.com and potentially more websites are affected, especially those that offer a preview of the file by default.

@orschiro
Copy link

orschiro commented Nov 9, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants