From 6d244cd236c0a83f2a2d9e7a49529a1b723d185d Mon Sep 17 00:00:00 2001 From: Jonas Platte Date: Tue, 9 Nov 2021 22:12:23 +0100 Subject: [PATCH] federation-api: Add note about minimum_valid_until_ts field type --- .../src/discovery/get_remote_server_keys_batch/v2.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/crates/ruma-federation-api/src/discovery/get_remote_server_keys_batch/v2.rs b/crates/ruma-federation-api/src/discovery/get_remote_server_keys_batch/v2.rs index ceefdc38..04cbc67e 100644 --- a/crates/ruma-federation-api/src/discovery/get_remote_server_keys_batch/v2.rs +++ b/crates/ruma-federation-api/src/discovery/get_remote_server_keys_batch/v2.rs @@ -64,6 +64,9 @@ pub struct QueryCriteria { /// /// If not supplied, the current time as determined by the notary server is /// used. + // This doesn't use `serde(default)` because the default would then be + // determined by the client rather than the server (and it would take more + // bandwidth because `skip_serializing_if` couldn't be used). #[serde(skip_serializing_if = "Option::is_none")] pub minimum_valid_until_ts: Option, }