Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Update Qlogo.cn.xml #7586

Merged
merged 2 commits into from
Nov 26, 2016
Merged

Conversation

gloomy-ghost
Copy link
Collaborator

No description provided.

@jeremyn
Copy link
Contributor

jeremyn commented Nov 15, 2016

I'm getting HTTP 400 for the URLs I checked in Qlogo.cn.xml. I'm not sure what to do about that.

I'm also not sure what to do about Tencent_CN.xml if it requires me to have an exit point in the GFW to test.

@gloomy-ghost
Copy link
Collaborator Author

Do you mean all URLs? They are CDN subdomains and most of them have blank roots.

Could you try to use Tor and set CN as exit node? I guess CN nodes are mostly honeypots but they should be okay for testing as long as they don't drop 443.

@gloomy-ghost
Copy link
Collaborator Author

I've added testing URLs for them.

Also would it be okay if @ivysrono could provide a screen recording to prove that these subdomains are available in https in RPC?

@ivysrono
Copy link
Contributor

dldir1
dp3
mmbiz
wx

@jeremyn
Copy link
Contributor

jeremyn commented Nov 16, 2016

@gloomy-ghost I'm having trouble with Tor. I added ExitNodes {CN} to my torrc, but it seems to hang halfway through Connecting to the Tor network.

Also, I'm not going to sign off on a pull request based on a screenshot, meaning no disrespect to @ivysrono .

@Hainish : do you at the EFF have any tools available to test something from within the Chinese GFW?

@gloomy-ghost
Copy link
Collaborator Author

There is a public proxy in China: 1.82.216.134:80

@jeremyn
Copy link
Contributor

jeremyn commented Nov 18, 2016

I'm not comfortable using a random public proxy. I'm sorry if it seems like I'm being difficult.

@gloomy-ghost
Copy link
Collaborator Author

gloomy-ghost commented Nov 18, 2016 via email

@jeremyn
Copy link
Contributor

jeremyn commented Nov 18, 2016

It's not a good precedent. An attacker could add a malicious ruleset by saying, "Here's my ruleset, and by the way, you need to test it through this specific proxy", where the attacker controls the proxy and manipulates the response. We should be able to check rulesets using our own independent tools and settings.

@gloomy-ghost
Copy link
Collaborator Author

gloomy-ghost commented Nov 18, 2016 via email

@jeremyn
Copy link
Contributor

jeremyn commented Nov 18, 2016

The whole problem here is that for some reason we get different results from exit points inside the GFW than we do from outside it, so the exit point does matter. I guess whatever technique is being used by the GFW could also be used by one malicious attacker managing one exit point.

@gloomy-ghost
Copy link
Collaborator Author

gloomy-ghost commented Nov 18, 2016 via email

@jeremyn
Copy link
Contributor

jeremyn commented Nov 18, 2016

So to properly test these domains, I have to proxy DNS, not just HTTP(S) traffic. I have to trust that both the proxy's DNS and web traffic match what a regular user inside the GFW sees, and if either DNS or web traffic differs, then I as a maintainer might be misled.

@jeremyn
Copy link
Contributor

jeremyn commented Nov 18, 2016

For "web traffic", I understand that HTTPS traffic can't be modified, however it's possible the person who controls the proxy might be able to route traffic outside the GFW, or not, depending on some criteria.

@gloomy-ghost
Copy link
Collaborator Author

To clarify my opinion, there is at least one official server supporting https when the domain works with a proxy. Network hijacking is everywhere, but if it only routes traffic, it's more likely a router.

I have to trust that both the proxy's DNS and web traffic match what a regular user inside the GFW sees

As I said, hijacking is everywhere and even Tencent can't say their https sites have 100% availability in China. CMCC is different with GWBN, CN2 is different with CHINANET. There are always some risks that https doesn't work, and we can't test them one by one. As a project rarely has official ruleset, we have to take this kind of risks.

@jeremyn jeremyn removed their assignment Nov 19, 2016
@jeremyn
Copy link
Contributor

jeremyn commented Nov 19, 2016

The way to guard against having the review manipulated is to use independent tools. In this case though I have to configure my environment a certain way and use one of a limited number of access points. I don't see how I personally could feel comfortable enough with a test in this situation to sign off on it. I've unassigned myself from this pull request.

I hope that @Hainish can come up with some approach. We need to figure out a general process for handling these inside-the-GFW issues.

@ivysrono
Copy link
Contributor

@gloomy-ghost 我看他们暂时解决不了GFW的测试问题,干脆绕开,进一步拆分,把没问题的先合并了。

@gloomy-ghost
Copy link
Collaborator Author

@ivysrono 现在好歹绑着Alexa排名高的域名,再拆就真没人处理拆出来的了…

@jeremyn
Copy link
Contributor

jeremyn commented Nov 24, 2016

@gloomy-ghost Here's an idea: since Tencent_CN.xml is the part with the difficult-to-test only-in-China problem, can you move it out of this pull request, maybe back into #6921 ?

@gloomy-ghost
Copy link
Collaborator Author

@jeremyn The problem is Travis-CI won't pass since some overlaps between two rulesets.

@jeremyn
Copy link
Contributor

jeremyn commented Nov 24, 2016

You mean t8.qlogo.cn? Can you remove only just that URL from the Tencent_CN.xml ruleset but otherwise leave it alone?

@gloomy-ghost
Copy link
Collaborator Author

Well, I used cherry-pick for reducing conflicts, but I will do it if it makes the progress quicker.

@jeremyn
Copy link
Contributor

jeremyn commented Nov 24, 2016

Please just only delete this line and make no other changes to Tencent_CN.xml, not even cosmetic changes.

@jeremyn
Copy link
Contributor

jeremyn commented Nov 25, 2016

I'm getting 400s for the tests associated with these targets:

@gloomy-ghost
Copy link
Collaborator Author

All of them are working for me.

PS: For q[1-4]?., you need to replace &

@jeremyn jeremyn merged commit 174a4ca into EFForg:master Nov 26, 2016
@jeremyn
Copy link
Contributor

jeremyn commented Nov 26, 2016

Okay, the URLs work now, thanks. I've updated the checklist. To clarify: by "replace &" you mean replacement it with &, so the URL should be

http://q.qlogo.cn/g?b=qq&k=Osib95YbszciblkflbYwNQgw&s=140

Merged.

@Hainish
Copy link
Member

Hainish commented Nov 29, 2016

I've reached out to some colleagues particularly well positioned to know about testing within the GFW. Hopefully they will have some tips on the best strategy here.

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

Successfully merging this pull request may close these issues.

4 participants