FragLand / FragLand/minestat

C#: Add port autodetection and make `port` parameter optional

Open
#238 0 comments 0 reactions 0 assignees View on GitHub
good first issue lang: C# new feature
Dominant language
C#
Stars
360
Forks
75
PR merge metrics
No merged PRs in 30d

Description

### Feature Description
We should have feature parity between different MineStat editions. The Python (and other) editions made the `port` parameter optional, if it can be automatically detected, e.g. through SRV records or default ports.

- [ ] Add port autodetection for default ports
- [ ] Make `port` parameter optional
- [ ] Document new usage patterns in README and docstring

> Originally proposed by @nimeng1299 in issue #237

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the C# edition's current port handling, then compare the Python edition's optional-port behavior and autodetection approach. Update the README and docstring with the new usage patterns; done means default-port detection works and callers can omit the port parameter.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
networking
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.