addCommand
timwilky opened this issue · 1 comments
timwilky commented
addCommand does not return a status, therefore there is no indication that use that would exceed the array bounds of the m_commands array has been dropped. Therefore a default maximum of only 8 commands can exist
When the dropped command is then requested, the failure page is then delivered.
quick fix to immediate issue, make the array size user defined in the header to stop dropped commands
unwiredben commented
open to pull requests on this one. I like the idea of making the array size a #define since that makes it more obvious.