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

option to put email in the config to share with server list #13461

Open
JasonTable opened this issue Apr 25, 2023 · 4 comments
Open

option to put email in the config to share with server list #13461

JasonTable opened this issue Apr 25, 2023 · 4 comments
Labels
Concept approved Approved by a core dev: PRs welcomed! Feature request Issues that request the addition or enhancement of a feature @ Startup / Config / Util

Comments

@JasonTable
Copy link

JasonTable commented Apr 25, 2023

Problem

Sometimes the owner(s) of a server on the public list have to be contacted.

Solutions

A way for server admin to put email in like how they can put a website url. this email can be kept private to the server list admins who can then use it in case they need to contact the server owner.

Alternatives

-

@JasonTable JasonTable added the Feature request Issues that request the addition or enhancement of a feature label Apr 25, 2023
@rubenwardy rubenwardy added the Concept approved Approved by a core dev: PRs welcomed! label Apr 25, 2023
@rubenwardy
Copy link
Contributor

I've been thinking that this should be done for a while, would be useful when we have security advisories too

@sfan5
Copy link
Collaborator

sfan5 commented Apr 25, 2023

I would rather do this by including contact details in the rejection message returned to the server and the owner would then contact us.
Doesn't cover the security advisory usecase obv.
Also worth noting that this requires something like luanti-org/serverlist#45 server-side.

@JasonTable
Copy link
Author

ooh yeah that might also be a good way to do it. maybe a ban reason can be provided as well. in those instances where a server is banned. so like in the log it could say:

<ERROR> 2023-04-25 14:52:31: [CurlFetch] HTTPFetch for servers.minetest.net/announce returned response code 403
<ERROR> 2023-04-25 14:52:31: [CurlFetch] Response body:
<ERROR> 2023-04-25 14:52:31: [CurlFetch] Banned (Server). [ban reason here] you may appeal by emailing [email protected]

something like that. perhaps a separate issue on the serverlist github should be made for that

@JasonTable
Copy link
Author

i would like to add that maybe an additional option should be to choose whether the email is public or private. if its private, only the server list owner can see, if its public, anyone can see. good way to get in contact with server owner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Concept approved Approved by a core dev: PRs welcomed! Feature request Issues that request the addition or enhancement of a feature @ Startup / Config / Util
Projects
None yet
Development

No branches or pull requests

4 participants