mirror of
https://github.com/mageddo/dns-proxy-server.git
synced 2026-04-25 09:25:56 +03:00
[GH-ISSUE #326] Auto configure as default DNS: Windows and OSX #115
Labels
No labels
bug
confirmed
discussion
duplicate
enhancement
feature
feature-request
not-planned
pull-request
secondary-feature
stale
triage
waiting-feedback
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/dns-proxy-server-mageddo#115
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @mageddo on GitHub (Feb 27, 2023).
Original GitHub issue: https://github.com/mageddo/dns-proxy-server/issues/326
Originally assigned to: @mageddo on GitHub.
Auto Configure
Be able to automatically configure DPS as the default DNS Server in the current OS and restore previous server when it shutdown
✅ Windows
=> Steps
=> Docs
How programatically set a DNS for windows
✅ Mac
Steps
Reference
TBD
@mageddo commented on GitHub (Mar 4, 2023):
Fixed for mac since DPS 3.7.0
@mageddo commented on GitHub (Mar 4, 2023):
Looks like all it's working for MAC I'm discussing at this thread #158
@mageddo commented on GitHub (Mar 6, 2023):
This a lot more difficult than I expected, can't use netsh because of netsh Windows language depending output.
Trying to use WMI instead, the JNA api to call WMI really sucks
@mageddo commented on GitHub (Mar 6, 2023):
Better try a new WMI client: https://github.com/freemansoft/jacob-project
@mageddo commented on GitHub (Mar 7, 2023):
Generated a version which comes with bundled dll: https://github.com/mageddo-projects/jacob-project
@mageddo commented on GitHub (Mar 8, 2023):
Rollback wmi based solution and used windows registry
@mageddo commented on GitHub (Mar 8, 2023):
CD failed https://github.com/mageddo/dns-proxy-server/actions/runs/4361050714/jobs/7624564619
@mageddo commented on GitHub (Mar 10, 2023):
It turns out that for some reason jna-platform dependency will broken arm64 native image generation because it's conflicting with quarkus somehow, so I need to think in some options to lead with this:
✔️ => 1 Remove quarkus dependency
Just remove Quarkus from the project and use some very basic http server to provide the statics and create the few rest apis, dependency injection can be replaced by dagger.
I think it will also reduce the size of the binary considerably and maybe simplify the complexity to build an static binary on the feature, see #285
=> 2 Wait quarkus issue to be solved
It can take a lot of time and the DPS release is broken while this, see https://github.com/quarkusio/quarkus/issues/31750 for the quarkus issue
=> Disclaimer
I don't realized another way to fix this issue as I really didn't get what on quarkus and jna make it happens, actually I know that JNA approach to load the libraries on a static field is bad approach when running on native-images but is something really difficult to fix, it requires a lot of refactoring
@mageddo commented on GitHub (Mar 10, 2023):
Okay, I'll decide this in some hours
@mageddo commented on GitHub (Mar 12, 2023):
Hope all is fixed now, waiting for the released
@mageddo commented on GitHub (Mar 13, 2023):
The issue is fixed, people can check on 3.9.2, for this new feature. I'm not closing this feature yet because I hope will be able to fix #306 in next hours then I would prefer to do a manual acceptance tests with the two features alive.