Skip to content

Commit

Permalink
Merge branch 'alex/fix-serialization-metric' into 'master'
Browse files Browse the repository at this point in the history
chore(crypto): set correct name for the vault's response serialization metric

Vault's serialization metric (message size & serialization time) erroneously uploads its data for the "response" label to the "request" label. This MR sets the right label. 

See merge request dfinity-lab/public/ic!15754
  • Loading branch information
altkdf committed Oct 30, 2023
2 parents 2e14994 + f47bf34 commit 9474fe8
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -247,7 +247,7 @@ impl SerializationObserver<Response<TarpcCspVaultResponse>> for CspVaultObserver
);
self.metrics.observe_vault_message_serialization(
ServiceType::Server,
MessageType::Request,
MessageType::Response,
domain,
method_name,
number_of_bytes,
Expand Down

0 comments on commit 9474fe8

Please sign in to comment.