JSONRPC: replace std::string with std::string_view when registring a new endpoint.

[brbzull0]

1 - Use std::string_view in the RPC manager class to avoid possible extra heap allocation when registring a new endpoint.
2 - Fix typo.

[bneradt]

Looks fine.

[SolidWallOfCode]

Looking at this -

return _handlers.emplace(name, std::move(call)).second;

What is the type of the name in the object? Does it persist after the string view goes out of scope?

[brbzull0]

Looking at this -

return _handlers.emplace(name, std::move(call)).second;

What is the type of the name in the object? Does it persist after the string view goes out of scope?

It does:

trafficserver/mgmt2/rpc/jsonrpc/JsonRPCManager.h

Line 237 in 62a596c

std::unordered_map<std::string, InternalHandler> _handlers; ///< Registered handler container.

std::string