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 DNSMatchDomains property to Interface (split DNS) #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fhriley
Copy link

@fhriley fhriley commented May 14, 2022

Based on the work of Stephen Larew [email protected].

This adds split DNS capability. MacOS and iOS don't fully support the separation of search domains and match domains so a best effort is made to implement to what the configuration intends.

Signed-off-by: Frank Riley [email protected]

Based on the work of Stephen Larew <[email protected]>.

This adds split DNS capability. MacOS and iOS don't fully support the separation of search domains and match domains so a best effort is made to implement to what the configuration intends.

Signed-off-by: Frank Riley <[email protected]>
@uberjay
Copy link

uberjay commented Sep 6, 2022

Thanks for putting this together -- it seems to behave exactly how I want it to... the question is, will it ever flow upstream?

@nicx
Copy link

nicx commented Sep 26, 2022

any chance that this PR will make it into the official release?

@aSilentSniper
Copy link

Please Merge this request so i dont have to build it myself

@evanphx
Copy link

evanphx commented Aug 31, 2023

Anything we can do to help get this merged? Would love to see it land.

@robertoschwald
Copy link

Still this badly needed PR is not merged. PLEASE MERGE, FOLKS!

@uberjay
Copy link

uberjay commented Apr 4, 2024

It's a bummer... I've been waiting for this before switching our company over to use WireGuard instead of OpenVPN... without it, the usability is not that great.

I continue to renew my apple developer subscription for the express purpose of building a custom WireGuard with this PR.

@philipstewart
Copy link

Keen to see split DNS on the iOS app, but it's worth noting the preference of the maintainer for contributions to this repository is either by sending properly formatted patches using git-send-email to the WireGuard mailing list, or via a prefixed branch in the official repository.

@robertoschwald
Copy link

I already sent a patch to the ml.

@philipstewart
Copy link

Wonderful - do you have a link? I could only find this one in a brief scan of the list, but the change proposed in this PR seems to be more extensive.

@robertoschwald
Copy link

Sent it April/5/24.

Is being held until the list moderator can review it for approval.

@thmo
Copy link

thmo commented Jul 8, 2024

Has it been approved - I don't see it here?

@adamcstephens
Copy link

Was this ever successfully posted to the mailing list? I'm not seeing it.

@robertoschwald
Copy link

I tried, but its waiting for moderation since ages...

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.

10 participants