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

enhanced whitelist #96

Open
cadeyrn opened this issue Dec 30, 2018 · 8 comments
Open

enhanced whitelist #96

cadeyrn opened this issue Dec 30, 2018 · 8 comments

Comments

@cadeyrn
Copy link
Owner

cadeyrn commented Dec 30, 2018

A whitelist has been added in Bookmarks Organizer 3.0. This includes a link to add bookmarks to the whitelist and the ability to reset the whitelist.

In a future update there will be an enhanced whitelist which lets you remove single entries from the whitelist.

@DuMuT6p
Copy link

DuMuT6p commented Nov 4, 2019

Hello,
Is there a way currently to view the whitelist ?

@cadeyrn
Copy link
Owner Author

cadeyrn commented Nov 5, 2019

No. This issue is about implementing the list view and the option to remove entries from the whitelist. You can only clear the whitelist in the add-on's settings in the current version.

@DuMuT6p
Copy link

DuMuT6p commented Nov 5, 2019

Thank you. This seems like a great feature to have.

@BUZZARDGTA
Copy link

BUZZARDGTA commented Sep 7, 2023

@cadeyrn Hello, I've read the source code from the whitelist and tried to apply a pull request with an export/import settings buttons.
While this is possible, and I managed to export successfully, I have noticed that you makes uses of bookmark nodes in order to whitelist.
In my opinion this is really wrong because if we change profile and whitelist the exact same bookmark, it will be another node (so it wont have any effect to import the whitelist on another profile).
Can you please consider re-writting this feature, and makes uses of the literal string URL instead of the node?

@BUZZARDGTA
Copy link

BUZZARDGTA commented Sep 7, 2023

Apply this whitelist change will be the first step to resolve any of the current whitelist feature requests:
#96: enhanced whitelist
#115: [request] whitelisting webpage domains before scan
#165: How to list all currently whitelisted bookmarks?
#182: [FEATURE REQUEST/QUESTION] Whitelisting source or destination address?
#202: [FEATURE REQUEST] - Add an option to export and import whitelist (settings)
#213: Problems with Facebook Bookmarks - Declared as "dead" - Allow whitelisting of domains
#217: Whitelist managing

@cadeyrn
Copy link
Owner Author

cadeyrn commented Sep 7, 2023

There are some things to consider:

  • Bookmarks Organizer always operated on the bookmark level. Does it really matches the user expectations if the whitelist does not operate on the bookmark level?
  • URLs can change over time. What would happen if the user changes the URL of the bookmark? Should it be removed from the whitelist or should the whitelist be modified if a bookmark change happens?
  • You can have more than one bookmark for the same URL. Do we really want to lose the ability to put one bookmark on the whitelist, but not the other one?
  • URLs are much longer than IDs, so much more data has to be saved. Does it have any meaningful impact for the storage size or the performance if many items are on the whitelist?

@BUZZARDGTA
Copy link

BUZZARDGTA commented Sep 7, 2023

Personally on my own project, that is in Python, I whitelist using:

[{HTTP_Code}] {Source URL} > [{HTTP_Code}] {Destination URL}
[301] https://www.google.com/ > [200] https://www.youtube.com/

(As many redirects as there are 👆)

And for "error" bookmarks
I use:
[{HTTP_code}] {URL}
[404] https://www.google.com

Exactly has you mentioned, it contains downsides as well.

🤷

@BUZZARDGTA
Copy link

I also make uses of regex in my whitelist, which is useful for certain websites who their URL changes every x time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants