> Status Execute(engine::Context &ctx, Server srv, Connection conn, std::string *output)

curious why they did pointer in-out param instead of references here. They could also just return these values as well? in-out param style is something I'd expect in C but not in modern C++.