Skip to content

Added Iran to domain fronting list#6006

Closed
pprucia wants to merge 1 commit intosignalapp:masterfrom
pprucia:master
Closed

Added Iran to domain fronting list#6006
pprucia wants to merge 1 commit intosignalapp:masterfrom
pprucia:master

Conversation

@pprucia
Copy link
Copy Markdown
Contributor

@pprucia pprucia commented Jan 2, 2017

First time contributor checklist

Contributor checklist

  • I am following the Code Style Guidelines
  • I have tested my contribution on these devices:
    I couldn't do it as I don't have an Iranian sim card. I checked whether Google.com is accessible through proxy server though. It is. Facebook and twitter are not, so I presume my proxy works correctly.
  • My contribution is fully baked and ready to be merged as is
  • I ensure that all the open issues my contribution fixes are mentioned in the commit message of my first commit using the Fixes #1234 syntax
  • I have made the choice whether I want the BitHub reward or not by omitting or adding the word FREEBIE in the commit message of my first commit

Description

I have added Iran country code to censorship configuration. As Google doesn't have local address there (google.ir) I use baseAndroid and baseGoogle urls only.

@moxie0
Copy link
Copy Markdown
Contributor

moxie0 commented Jan 2, 2017

Hmm, I was told that google.com is blocked in Iran. Were you testing on mobile data networks?

@pprucia
Copy link
Copy Markdown
Contributor Author

pprucia commented Jan 2, 2017

No. I only used Iranian proxy server and set it up with my wifi connection (from Poland). Check google.com, facebook and twitter. Only google worked. I have found some articles from 2013-2015 that Iran might allow Google in but none with definite answer.

I think this should be verified by someone there.

moxie0 pushed a commit that referenced this pull request Jan 3, 2017
@moxie0
Copy link
Copy Markdown
Contributor

moxie0 commented Jan 3, 2017

in 3.26.0

@moxie0 moxie0 closed this Jan 3, 2017
@pprucia pprucia deleted the master branch January 14, 2017 17:29
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.

2 participants