openmultiplayer / openmultiplayer/open.mp
More sanity checks for config options
Open
Nobody has claimed this yet.
enhancement
good first issue
- Dominant language
- C++
- Stars
- 641
- Forks
- 208
- PR merge metrics
- No merged PRs in 30d
Description
Some sanity checks could be added to warn about one of the examples:
maxnpcvariable exceeds the maximum number of player slotsqueryis disabled butannounceis enabled, so the server will not be visible in masterlist- More checks on negative values for the most of settings which cannot apply such numbers
Other checks which also can be applied from the server start:
- scriptfiles folder cannot be created for some reason (some scripts rely on this and may work incorrectly)
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the server-start configuration validation and the code that creates the scriptfiles folder. Trace how maxnpc, query, announce, and numeric settings are loaded, then define warnings for the listed invalid combinations and values, including folder-creation failure. Done means these startup conditions produce clear warnings without preventing valid server startup.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100