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

Create newtalk.tw.xml #7452

Merged
merged 10 commits into from
Dec 19, 2016
Merged

Create newtalk.tw.xml #7452

merged 10 commits into from
Dec 19, 2016

Conversation

ivysrono
Copy link
Contributor

No description provided.

@gloomy-ghost
Copy link
Collaborator

Timeout
admin.newtalk.tw

InvalidCert
www.newtalk.tw
3fe8l33fe8f3ff0f0oc3ff07fa0al2000.newtalk.tw
www.f38lo3f3fe8e3ff08c33fe8f3ff0f0al.newtalk.tw
l33fe8f34730ff0f896c0oc3ff0al.newtalk.tw
l3f7fd83ff0f0oc3ff07fa0al2000.newtalk.tw
lo3ff07fd83fe8c3fe83f46e13ff0f3fe808061al.newtalk.tw
lo47fd8e53c3fe83ff0al.newtalk.tw
lo47fd8e53c3ff0a3ff0l.newtalk.tw
lo73fe83ff0fd83fe8c3fe83f3ff0f0a3fe8l.newtalk.tw
www.lo73ff0f3ff0d83fe8c3fe83f3ff0f0a3fe8l.newtalk.tw
lo7f3ff0d83fe8c3fe83f3f7fd8f0f0al.newtalk.tw
lo7fd3fe883fe8c3f3ff0f0al.newtalk.tw
loa047fd8e53c3ff0a1000l.newtalk.tw
local.newtalk.tw
m.newtalk.tw
mail.newtalk.tw
www1.newtalk.tw

@gloomy-ghost
Copy link
Collaborator

www is in SANs now.

Also found a CDN domain: https://s.ntkstatic.org/media/blog/293/blog_img_293.jpg

@gloomy-ghost gloomy-ghost self-assigned this Dec 12, 2016
@gloomy-ghost
Copy link
Collaborator

话说那坨奇怪的域名好像都重定向到^

@ivysrono
Copy link
Contributor Author

试了下,那一大坨奇怪域名都SANs了,http会重定向到^,https就不跳转了。
这个怎么处理?直接https?

@gloomy-ghost
Copy link
Collaborator

还是有不在的,比如说 https://www.ssllabs.com/ssltest/analyze.html?d=lo3ff07fd83fe8c3fe83f46e13ff0f3fe808061al.newtalk.tw

我觉得注释里说一下这是泛解析,然后把这些都直接重定向到^吧…

@ivysrono
Copy link
Contributor Author

大爷的,还发现了坨无限域名:nvk\d*.
我还要去处理渣浪的……

@gloomy-ghost
Copy link
Collaborator

gloomy-ghost commented Dec 12, 2016 via email

@ivysrono
Copy link
Contributor Author

ivysrono commented Dec 12, 2016

我说怎么那么困惑,原来是代理的原因……

@ivysrono
Copy link
Contributor Author

见鬼,过去7天才看到这条评论……

<test url="http://ap1.newtalk.tw/" />
<test url="http://www.newtalk.tw/" />

<rule from="^http://\w+\.newtalk\.tw/" to="https://newtalk.tw/" />
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

\w+ doesn't match with dash, such as abcdefg-123456789.newtalk.tw. I'd suggest you to use .+.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我就记得哪次中招过,.+ 通不过测试

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.+ 会要求两个额外的测试

Copy link
Contributor Author

@ivysrono ivysrono Dec 19, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://travis-ci.org/EFForg/https-everywhere/jobs/185161024#L193

failure: /opt/utils/../src/chrome/content/rules/newtalk.tw.xml failed test: The 'from' rule contains unescaped period in regular expression. Try escaping it with a backslash.

这个是多加两个测试就会通过?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

丧心病狂的电信……直接用手机开热点了

之前还没注意过/utils/trivial-validate.py#L74-L84,看了下觉得这个检测方式的误报率有点高……

留着\S吧,顺手 git blame Yimg.jp.xml (逃

@gloomy-ghost gloomy-ghost merged commit 7994b75 into EFForg:master Dec 19, 2016
@gloomy-ghost gloomy-ghost removed their assignment Dec 19, 2016
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.

2 participants