Noticed that we're using type Error = serde_json::Error in FromStr, but we map it everywhere to tonic::Status.
Refactored this to map it only in one place and avoid adding another mapping in the next diff

Depends on D13222

cargo build -p identity