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

fix: clash wireguard config pre-shared-key key name wrong #283

Merged
merged 3 commits into from
Nov 20, 2024

Conversation

Zoupers
Copy link
Contributor

@Zoupers Zoupers commented Nov 16, 2024

According to config doc, the wg presharedkey config name should be pre-shared-key, not preshared-key.

Copy link

netlify bot commented Nov 16, 2024

Deploy Preview for surgio-documentation failed.

Name Link
🔨 Latest commit f43be62
🔍 Latest deploy log https://app.netlify.com/sites/surgio-documentation/deploys/673e0996a63a2a000899e11f

@Zoupers
Copy link
Contributor Author

Zoupers commented Nov 16, 2024

According to this issue, this change might be not compatiable with previous version of clash, so we might need to add new configuration file for mihomo ?

@geekdada
Copy link
Member

@Zoupers What happens when both keys are provided?

@Zoupers
Copy link
Contributor Author

Zoupers commented Nov 19, 2024

That's a great idea! I believe there won't be any issues. Let me improve the code.

@geekdada
Copy link
Member

Please verify that locally first because Clash verifies the file for unreadable properties.

@Zoupers
Copy link
Contributor Author

Zoupers commented Nov 20, 2024

I changed the way to achieve the goal, now we need to specify clashCore to have pre-shared-key. So there will be no need to test clash, just clash-meta.
And I have tested locally, it works fine.

@geekdada
Copy link
Member

非常感谢贡献代码!

@geekdada geekdada merged commit 5b9785e into surgioproject:master Nov 20, 2024
5 of 9 checks passed
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