Skip to content

[Draft] Relay support#130

Draft
flleeppyy wants to merge 1 commit into
spacestation13:masterfrom
flleeppyy:relay-support
Draft

[Draft] Relay support#130
flleeppyy wants to merge 1 commit into
spacestation13:masterfrom
flleeppyy:relay-support

Conversation

@flleeppyy
Copy link
Copy Markdown

@flleeppyy flleeppyy commented Apr 7, 2026

This PR introduces relay config options for servers. Please bear with me as I have not worked with Vue very much before, I am more acquaint to React. My brain really hurts.

This stems from someone mentioning the join link doesn't work, pooba didn't PR the config changes correctly, play.monkestation.com is a direct IP domain which we have no web server setup for and is just for direct connections. hub.monkestation.com is our server hub

It "works", it rerenders the parent elements for some reaso unnecessarily and i dont know why

firefox_svq73zAzHl.mp4

@flleeppyy flleeppyy requested a review from Wirewraith as a code owner April 7, 2026 05:04
@flleeppyy flleeppyy marked this pull request as draft April 7, 2026 05:04
@Wirewraith
Copy link
Copy Markdown
Contributor

I'm unsure of the purpose of this relay system. If you just want buttons that work, it seems like you could just do something like:

	"links": [
		{
			"title": "US East — Ashburn, VA",
			"ip": "useast.monkestation.com",
			"port": 1234
		},
		{
			"title": "Europe — Warsaw, PL",
			"ip": "warsaw.monkestation.com",
			"port": 1234
		},
		{
			"title": "Europe — Gravelines, FR",
			"ip": "gravelines.monkestation.com",
			"port": 1234
		}
	]

Please let me know if I'm misunderstanding.

@ZeWaka
Copy link
Copy Markdown
Member

ZeWaka commented Apr 7, 2026

I'm unsure of the purpose of this relay system.

They have it for better geo-routing using a different backend since some ISPs are trash.
I do agree that they could just set the link entries, however.

@flleeppyy
Copy link
Copy Markdown
Author

flleeppyy commented Apr 7, 2026

I'm unsure of the purpose of this relay system.

They have it for better geo-routing using a different backend since some ISPs are trash. I do agree that they could just set the link entries, however.

It would mean we would have a total of 8 link entries which seems like a lot compared to the other servers, and it takes up a lot of page space, and it means 2 more links every relay we add

@ZeWaka
Copy link
Copy Markdown
Member

ZeWaka commented Apr 12, 2026

It would mean we would have a total of 8 link entries which seems like a lot compared to the other servers, and it takes up a lot of page space, and it means 2 more links every relay we add

Is there really a reason the relays need to be on this page?

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.

3 participants