Closed teooko closed 1 month ago
This will not work, no. Did you test it? Generally a good idea before you PR something.
Unless specifically included options like this will get stripped out. There is very little interest on the FR, so we can reevaluate if that changes but this will likely need to be more explicit as well for security, not just allowing any arbitrary set of arguments.
thanks for your interest in the project
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion for related concerns. See our contributing guidelines for more details.
Proposed change
I reviewed this feature request and tried to add custom arguments to the gamedig query according to this format. I am unsure if this is the proper way of doing it, but I would appreciate some guidance if necessary.