https://gitlab.synchro.net/main/sbbs/-/merge_requests/541#note_7269
Ah, I was hoping Deuce would add his two cents. He has security and usability concerns around user-supplied data potentially used in query parameters. He said this should be 3-5 classes, not one.
The problem with committing the way it is, is: scripts likely would be written to target this API and then we'd have to continue to maintain that backward compatibility (or hear about it from Nelgin). :-) It's better to get the API right the first time, if we can.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net