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

Switch to wormhole servers from Least Authority more easily #125

Closed
2 tasks done
Jacalz opened this issue Oct 27, 2023 · 5 comments
Closed
2 tasks done

Switch to wormhole servers from Least Authority more easily #125

Jacalz opened this issue Oct 27, 2023 · 5 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@Jacalz
Copy link
Owner

Jacalz commented Oct 27, 2023

Checklist

  • I have searched the issue tracker for open issues that relate to the same feature, before opening a new one.
  • This issue only relates to a single feature. I will open new issues for any other features.

Is your feature request related to a problem?

The folk over at Least Authority have Winden and Destiny as some very good clients for sending using the same Wormhole protocol. This is very useful because they allow to send and revive from a web browser and Android/iOS respectively. However, they do not use the same servers by default and a user would have to manually use these settings:

  • AppID: lothar.com/wormhole/text-or-file-xfer
  • Rendezvous URL: wss://mailbox.mw.leastauthority.com/v1
  • Transit Relay Address: relay.mw.leastauthority.com:4001

Describe the solution you'd like to see.

I'm thinking that we can update the advanced section to use widget.SelectEntry{} instead of widget.Entry{} and provide the Least Authority servers as alternatives without the user having to paste them in manually (also makes it easier to switch between that and the standard settings):

image

Even though the AppID parameter is the same, I suppose we could add only lothar.com/wormhole/text-or-file-xfer to the options there. I suppose it would be better to also stop putting the defaults in the placeholder text and just have them all in the options list instead.

@Jacalz Jacalz added enhancement New feature or request good first issue Good for newcomers labels Oct 27, 2023
@Jacalz Jacalz added this to the v3.6.0 milestone Oct 27, 2023
@Jacalz Jacalz changed the title Allow to more easily switch to wormhole servers from Least Authority Switch to wormhole servers from Least Authority more easily Oct 27, 2023
@Jacalz
Copy link
Owner Author

Jacalz commented Oct 27, 2023

@GalihFajar Is this something you perhaps would be interested in looking at? I've tried to make the problem and solution as clear as possible :)

@GalihFajar
Copy link

@Jacalz Sure, I'd love to take a look at it. Will be slow response on the weekend though

@Jacalz
Copy link
Owner Author

Jacalz commented Oct 27, 2023

Cool. Thanks. That's no problem at all. Take all the time you need. I'll assign you to the issue for now

@GalihFajar
Copy link

Hi @Jacalz, I've made the PR for this, but honestly I'm not quite sure whether it switches properly from the default ones. Can you help to take a look and verify? Many thanks.

@Jacalz
Copy link
Owner Author

Jacalz commented Oct 29, 2023

Works great. Thanks 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants