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

[Bug]: ShadowTLS parameter issues #267

Open
radiumatic opened this issue Aug 10, 2024 · 1 comment
Open

[Bug]: ShadowTLS parameter issues #267

radiumatic opened this issue Aug 10, 2024 · 1 comment
Labels
bug Something isn't working to deploy Considerable for implementation

Comments

@radiumatic
Copy link

Describe the bug
A clear and concise description of what the bug is.

  1. The handshake server port doesn't get added to the configuration
  2. The port value in the "add handshake server" feature gets added to the configuration as str, not int.

To Reproduce
Steps to reproduce the behavior:

  1. Create a ShadowTLS inbound and set the handshake server.
  2. Add a shadowsocks server and add it as the detour to the ShadowTLS server.
  3. Create a chain config in either NekoRay or NekoBox and put the ShadowTLS first, then the shadowsocks config.
  4. You can see in the server logs that it tries to connect to [handshake_server]:0 instead of whatever port you specified and the port isn't present in the sing-box config.

Expected behavior
A clear and concise description of what you expected to happen.

Ports being added correctly.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@radiumatic radiumatic added the bug Something isn't working label Aug 10, 2024
@alireza0 alireza0 added the to deploy Considerable for implementation label Aug 12, 2024
@alireza0
Copy link
Owner

Thank you for your report. I will take a look and fix it soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working to deploy Considerable for implementation
Projects
None yet
Development

No branches or pull requests

2 participants