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

Replace the Vietnamese filter in stock lists #2600

Closed
wants to merge 1 commit into from
Closed

Replace the Vietnamese filter in stock lists #2600

wants to merge 1 commit into from

Conversation

NVHT
Copy link

@NVHT NVHT commented May 9, 2017

The filter in the lists is outdated & not supported by Fanboy anymore.

@Snapy
Copy link

Snapy commented May 9, 2017

You should use https for supportURL.

@NVHT
Copy link
Author

NVHT commented May 9, 2017

The support site doesn't use or have https protocol.

@bershan2
Copy link
Contributor

@connor37 I went to https://abpvn.com/ and it loaded fine, although there is an (i) next to the URL bar (I use Chrome). Looks like the site itself has a SSL certificate, but has Non-Secure dependencies http://i.imgur.com and http://fonts.googleapis.com... but those do have SSL and even default to SSL!
Looks like a simple fix for the site admin: just replace http:/ with https:/ in every link. Do you know how to contact them?

@NVHT
Copy link
Author

NVHT commented May 10, 2017

You can contact them through email: support@abpvn.com or through Facebook page: https://www.facebook.com/abpvn.org.

@bershan2
Copy link
Contributor

bershan2 commented May 10, 2017

@connor37 I contacted them on Facebook, it showed that they saw the message, but did not reply...

Site is reachable via HTTPS, it just is not validated as HTTPS. All content is served via HTTPS: the dependencies default to HTTPS, so the connections are upgraded right after the initial handshake. Thus I propose to hard-code support URL to https:// and nag the developers later.

Update: They did reply and stated that they can't upgrade to HTTPS because CloudFlare SSL does not support Windows XP. Here is the explanation:

We [are] current using cloudflare and cloudflare ssl [does] not support windows xp but we need [to] support anyone so we can't redirect all access to https
We support both https and http. You also [should] update pull request support url to https[.] thanks. But default we will not redirect http request to https. https://abpvn.com/ https supported by cloudflare

Update assets.json

replace the Vietnamese filter in stock lists
@NVHT
Copy link
Author

NVHT commented May 12, 2017

@bershan2 All commits squash.

@bershan2
Copy link
Contributor

Update: The supportURL site only partially supports HTTPS: the main page does support HTTPS, but some pages it links to default to HTTP: e.g., abpvn.com/ie that serves Internet Explorer users.

@gorhill Do you still consider adding this list to uBO standard subscription list (either replacing "Fanboy's Vietnamese" or adding it along side)? If so, I'll continue bugging the site owner about enabling HTTPS on other pages too.

@NVHT
Copy link
Author

NVHT commented May 25, 2017

The owner actually have the github repo at https://github.com/abpvn/abpvn, I can change the supportURL to this one.

@bershan2
Copy link
Contributor

@connor37 The supportURL should lead people to the place they will get the help. In this case, it is the https://abpvn.com, which has some general information and a link to Facebook -- the author's preferred place for communication. The GitHub repo has only 3 issues so far.

@bershan2
Copy link
Contributor

bershan2 commented Jun 19, 2017

@connor37
UPDATE: these concerns turned out to be false, see below comments!

Since this list hasn't been added yet, I just looked at its site and have a few questions/concerns about it:

  1. Under the help section the "Free support" link, "ỦNG HỘ MIỄN PHÍ", leads to "get easy money fast" Facebook note... weird...
  2. Under the help section the "donate" (paid support) page, "ỦNG HỘ THẺ CÀO NHẬN LINK VIP", does not support HTTPS... and asks for credit card info, as far as I can tell. (I did not notice this at first a month ago or they changed something.)
  3. The subscription link is randomly generated on a special page https://abpvn.com/get, while the list is hosted on GitHub https://github.com/abpvn/abpvn/blob/master/filter/abpvn.txt. Why would privacy-minded group track their users? Or is this just active user counter?
  4. https://filterlists.com does not have them in the table. I'll file an issue there to add it.
  5. The list is very short, I don't think it can cover the entire country's web.

@NVHT
Copy link
Author

NVHT commented Jun 19, 2017

  1. Donation by using the referral code in the app from the note, it is not about how to get easy money fast.
  2. Yeah, I noticed that some of the pages don't support HTTPS, but this donate page doesn't ask for credit card info. You donate to the list maintainer through scratch cards for adding credits to your mobile plan.
  3. I really don't know about this.
  4. Great, this filter is support by Adguard.
  5. From my experiences, the list can block any ads from any Vietnamese site, except for some sites required specific filters.

@bershan2
Copy link
Contributor

bershan2 commented Jun 19, 2017

@connor37 Thanks for explanation!
Sorry for that, I used Google translate (which does not work that great). I apologize for baseless suspicion.

  1. and 2. That's an interesting (clever) way to donate, I'll look in it more. 👍
  2. see above.
  3. Probably we should use the GitHub URL, and tell the author that uBlock Origin and filterlists.com will remain uncounted for now.
  4. filterlists.com had added this https://github.com/collinbarrett/FilterLists/commit/49edd36cc87a8e512246aefb37c68d97b9adad7f.
  5. I'm glad to hear that it is very efficient.

@NVHT
Copy link
Author

NVHT commented Sep 28, 2017

@gorhill Any thoughts on this so far?

@gorhill
Copy link
Owner

gorhill commented Sep 28, 2017

The commit is wrong, I have to refuse it:

  • The contentURL is wrong, it does not point to the list itself, but rather to a GitHub HTML page from where the list can be viewed. This has to be the raw list.
  • We must not replace an existing list, unless it's just a different URL for the same list.
    • The list proposed here is from a different maintainer.
    • There virtually no overlap between the list proposed here and Fanboy Vietnamese

The proper fix it to create a new entry for VIE, and remove Fanboy Vietnamese from stock filter list, since it's now labelled as "Officially unsupported".

I will do this.

@gorhill gorhill closed this Sep 28, 2017
gorhill added a commit that referenced this pull request Sep 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants