Skip to content

fix: respect merged UDP option in Clash export#54

Open
slackworker wants to merge 1 commit into
MetaCubeX:masterfrom
slackworker:fix/clash-udp-override
Open

fix: respect merged UDP option in Clash export#54
slackworker wants to merge 1 commit into
MetaCubeX:masterfrom
slackworker:fix/clash-udp-override

Conversation

@slackworker
Copy link
Copy Markdown

Summary

  • Use the merged udp option when exporting Clash proxies instead of the raw node field.
  • Ensure request/global udp=true|false overrides are reflected in final Clash output.
  • Keep Snell behavior unchanged by continuing to omit the UDP field for Snell nodes.

Why

The previous implementation wrote x.UDP directly in the final Clash node output, which bypassed the already-merged effective udp value and prevented expected request-level override behavior.

Test Plan

  • Reproduced the issue with a running local subconverter service.
  • Verified behavior around Clash output generation path after the fix.
  • Build was not completed locally due missing dependencies (CURL for full CMake build and yaml-cpp for static build).

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant