match: more string_view improvements
match: more string_view improvements

  - It has been suggested that `string_view` is better than `const
    string_view&`[1].  Switch all calls to that type.

  - In some cases the optimizations using string_view allowed
    non-zstrings to be passed to sdbus calls without compiler warning.
    Revert back to sacrificing the slight performance of constructing
    the string for safety.

[1]: https://quuxplusone.github.io/blog/2021/11/09/pass-string-view-by-value/

Signed-off-by: Patrick Williams <cbb7353e6d953ef360baf960c122346276c6e320@stwcx.xyz>
Change-Id: I21a542fc9264b0f998e1c234064b6e6414327b82

openbmc/sdbusplus,openbmc/sdbusplus,openbmc/sdbusplus,openbmc/sdbusplus