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

Please add support for quic:// URIs and multiple device addresses #893

Closed
wararjey opened this issue May 14, 2022 · 3 comments
Closed

Please add support for quic:// URIs and multiple device addresses #893

wararjey opened this issue May 14, 2022 · 3 comments
Assignees
Milestone

Comments

@wararjey
Copy link

Description of the issue

Hello,
I am using your syncthing-fork on my android devices. syncthing started to support quic:// URIs somewhat recently. It seems that the frontend does not support them, yet. I do run my own discovery server, so static addressess are not strictly necessary. However, for the sake of robustness and reliability, I do set the introducer's address statically in all my other syncthing clients. The introducer does not run on an android device, btw.

Reproduction Steps

  1. Have a syncthing peer that is reachable at sync.yourdomain.net:someport. Set the introducer's listening address to something like
    "tcp://:someport, quic://:someport"
  2. Install syncthing fork and add the introducer device.
  3. Try to change the introducer's address to something like
    "quic://sync.yourdomain.net:someport" or
    "quic://sync.yourdomain.net:someport, tcp://sync.yourdomain.net:someport, dynamic"
  4. The frontend will reject the devices's address as invalid, although they are valid in modern syncthing backend versions.

Workaround is to change the address options over the web interface. However, it would be useful if one could change it directly in the frontend.

Version Information

  • App Version: 1.20.1
  • Syncthing Version: v1.20.1
  • Android Version: various
  • Device manufacturer: various
  • Device model: various

Device platform info

Imho, I do not think that is revelevant for this issue.

Android Log

Imho, I do not think that is revelevant for this issue.

Thank & Regards,

@Catfriend1 Catfriend1 added this to the v1.20.4 milestone Jul 13, 2022
@Catfriend1 Catfriend1 self-assigned this Jul 13, 2022
@Catfriend1 Catfriend1 modified the milestones: v1.20.4, v1.20.3 Jul 13, 2022
@Catfriend1
Copy link
Owner

Does release https://github.com/Catfriend1/syncthing-android/releases/tag/v1.20.3.1 fix the problem for you? Please report back.

@nicolasmeric
Copy link

It works for me.
Thanks.

@wararjey
Copy link
Author

Yes it does, thank you :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants