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

ensure Subscribe links work #252

Closed
elypter opened this issue Feb 18, 2018 · 6 comments
Closed

ensure Subscribe links work #252

elypter opened this issue Feb 18, 2018 · 6 comments
Labels
bug something isn't working web front-end user interface

Comments

@elypter
Copy link

elypter commented Feb 18, 2018

tested with nano adblocker (ublock origin fork)

@collinbarrett
Copy link
Owner

hmm...

So, currently on the beta, subscribe links are rendered like:
abp:subscribe?location=http%3A%2F%2F1hosts.cf%2F&title=1Hosts

Using the ABP link generator, the same link gets generated like:
abp:subscribe?location=http%3A%2F%2F1hosts.cf%2F&title=1Hosts

So, I think the encoded version of the ampersand is correct.

That being said, there have been issues with making sure FilterLists links are recognized by the extensions. The current beta version of the site seems to work hit-or-miss with subscribe links. I'm going to keep this issue open to address the broader scope of trying to ensure subscribe links work on the new version of the site. If anyone comes across this and has issues with subscribe links not working, please report below browser version, os version, which extension you are using, etc. Will try to fix.

More Info: #79 (comment)

@collinbarrett collinbarrett added the feedback wanted provide your input label Feb 18, 2018
@collinbarrett collinbarrett changed the title surplus "&amp" in beta subscription urls make subscription fail ensure Subscribe links work Feb 18, 2018
@collinbarrett collinbarrett added the web front-end user interface label Feb 18, 2018
@elypter
Copy link
Author

elypter commented Feb 19, 2018

it works now. looks like beta and stable site versions are the same now. if i stumble upon a list that doesnt work ill report it.

@collinbarrett collinbarrett added invalid doesn't seem right and removed feedback wanted provide your input labels Feb 19, 2018
@collinbarrett
Copy link
Owner

links seem to work on various browsers/extensions I've tested. I'm going to go ahead and close for now, but let me know if you find a story where subscribe links fail.

@gwarser
Copy link
Contributor

gwarser commented Sep 14, 2020

@collinbarrett you should probably revisit this & escape. Sample link on https://adblockplus.org/en/abp_linkfactory and subscriptions on https://adblockplus.org/en/subscriptions all use &. uBO subscriber was rewritten to URL-parse subscription links and now require special handling for filterlists.com - gorhill/uBlock@714f07d

@collinbarrett
Copy link
Owner

re-opening per this. I'm not sure how to fix just yet. not sure if it's a FilterLists.com issue. A uBO issue. A Chrome issue. Or something else...

@collinbarrett collinbarrett reopened this Sep 14, 2020
@collinbarrett collinbarrett added bug something isn't working and removed invalid doesn't seem right labels Sep 14, 2020
@collinbarrett
Copy link
Owner

collinbarrett commented Sep 14, 2020

I think the remaining intermittent issue is a uBO issue that is fixed in the latest dev build. Closing this for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something isn't working web front-end user interface
Projects
None yet
Development

No branches or pull requests

3 participants