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

Add Carrier Services to OpenVPN whitelist #31

Conversation

MarkIannucci
Copy link
Contributor

... to allow RCS messages to work

... to allow RCS messages to work
@MarkIannucci
Copy link
Contributor Author

Fixes #30 .

@MarkIannucci
Copy link
Contributor Author

I am having intermittent issues with RCS messaging. Will need to do some additional testing.

Copy link
Owner

@rajannpatel rajannpatel left a comment

Choose a reason for hiding this comment

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

This is a smart find!

@rajannpatel
Copy link
Owner

jibecloud.net is not blocked inside Pi-Hole, those DNS queries are passing through just fine. I am going to whitelist all the apps you mentioned:

  • Google Play services
  • Messages
  • MmsService
  • Carrier Services

And see where we land.

@rajannpatel
Copy link
Owner

I have the option to whitelist "com.android.service.ims.RcsServiceApp" also... I will try adding that to the mix

@MarkIannucci
Copy link
Contributor Author

MarkIannucci commented Mar 19, 2019

And the interesting thing is... I think the intermittent problem I found was on my recipient's end not mine. So I'm good with you completing this PR and merging.

I don't have the com.android.service.ims.RcsServiceApp in my list in OpenVPN.

And as I'm thinking about this more, lets let this bake a bit before finalizing the list and merging.

@MarkIannucci MarkIannucci reopened this Mar 19, 2019
@rajannpatel rajannpatel merged commit 5ee6975 into rajannpatel:master Mar 19, 2019
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.

None yet

2 participants