Skip to content

PR related to the #520 issue#2261

Open
top-master wants to merge 2 commits into
SoftEtherVPN:masterfrom
top-master:master
Open

PR related to the #520 issue#2261
top-master wants to merge 2 commits into
SoftEtherVPN:masterfrom
top-master:master

Conversation

@top-master
Copy link
Copy Markdown

Implements DNS-leak solution mentioned by #520 issue.

Better late than never, however, feel free to remove the AGENTS.md file.

@chipitsine
Copy link
Copy Markdown
Member

vibe coding these days often hallucinate and produce AI slope.
maybe in 5-10 years it will change

@chipitsine
Copy link
Copy Markdown
Member

also, I would keep libhamcore as a module

@top-master
Copy link
Copy Markdown
Author

top-master commented Apr 20, 2026

Nothing in this PR was Vibe-coded, however, I Vibe-merged my changes from another folder into this repo's latest commit. Anyhow, the Github actions and/or workflows seem to pass.

I mean, the only issue I could find was un-related to my changes, and instead says:

'vpnsetup' is not recognized as an internal or external command,
operable program or batch file.

@chipitsine
Copy link
Copy Markdown
Member

what I suggest is to get it properly working in your fork and open a PR once ready

@top-master
Copy link
Copy Markdown
Author

top-master commented Apr 21, 2026

Everything seems to be merged without changing SoftEther's logic, meaning, just like I asked the AI-agent to, only the minimal lines needed for this are merged.

However, the AI-Agent did "improve" a little of my own code, which I reverted in a separate commit. And the detiled comments should prevent future improvements.

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