Create 5ch.xml by maboroshin · Pull Request #19126 · EFForg/https-everywhere · GitHub
Skip to content
This repository was archived by the owner on Nov 6, 2023. It is now read-only.

Create 5ch.xml#19126

Merged
cschanaj merged 8 commits into
EFForg:masterfrom
maboroshin:master
Sep 8, 2020
Merged

Create 5ch.xml#19126
cschanaj merged 8 commits into
EFForg:masterfrom
maboroshin:master

Conversation

@maboroshin

@maboroshin maboroshin commented Apr 7, 2020

Copy link
Copy Markdown
Contributor

List related ruleset

Currently 2ch.net is being redirected to 5ch.net.

P.S. ( September 8, 2020)

  • 1st test group: Non-forum server: relatively easy to be unused server.
  • 2nd test group: Forum server

Checked on September 8, 2020 at here

Comment thread src/chrome/content/rules/5ch.xml Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please do not use wildcard targets if possible.

@maboroshin maboroshin Apr 7, 2020

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Is this better?
<target host="\w+.5ch.net" />

5ch (formerly 2ch) is the largest forum in Japan. 5ch has fewer servers. But it's fluid rather than fixed.

The former 2ch (not 5ch) has grown to over 100 subdomains: sample

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The host attribute is not a regex, so this won't work. If it's actually impossible to list all subdomains, a wildcard is acceptable.

@maboroshin maboroshin Aug 19, 2020

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think a wildcard is good.

Perhaps they are worried about connection errors with HTTPS. But, SSL has been used since the previous 2ch (current 5ch). The web operator is SSL-enabled, can also connect to both via HTTP. It won't cause any problems.

On the other hand, the official smartphone loading program (Ajax, still output HTTP) has been stalled in development. Programmers are not active. And external third party web tools also.

@zoracon zoracon added the top-1k label Apr 11, 2020
@pipboy96 pipboy96 mentioned this pull request May 23, 2020

@maboroshin maboroshin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Move this comment to the above. (missed)

@cschanaj cschanaj left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for your PR. Travis did not pass because you missed some trailing slashes for the test URLs. You may apply the suggestions to fix this. Besides, would you please use tabs for indentations as suggested by our contributing guide ?

Comment thread src/chrome/content/rules/5ch.xml Outdated
Comment thread src/chrome/content/rules/5ch.xml Outdated
Comment thread src/chrome/content/rules/5ch.xml Outdated
@cschanaj cschanaj self-assigned this Sep 7, 2020
Comment thread src/chrome/content/rules/5ch.xml Outdated

@maboroshin maboroshin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@cschanaji Thanks for the assist.
Seems like a good change. Now we just have to wait for the merge?

@cschanaj

cschanaj commented Sep 7, 2020

Copy link
Copy Markdown
Collaborator

maboroshin and others added 5 commits September 8, 2020 19:40
Co-authored-by: Chan Chak Shing <cschanaj@users.noreply.github.com>
Co-authored-by: Chan Chak Shing <cschanaj@users.noreply.github.com>
Co-authored-by: Chan Chak Shing <cschanaj@users.noreply.github.com>
Co-authored-by: Chan Chak Shing <cschanaj@users.noreply.github.com>
Comment thread src/chrome/content/rules/5ch.xml Outdated

@cschanaj cschanaj left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Otherwise, LGTM

Comment thread src/chrome/content/rules/5ch.xml Outdated
Co-authored-by: Chan Chak Shing <cschanaj@users.noreply.github.com>
@cschanaj cschanaj merged commit 13dfd7d into EFForg:master Sep 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants