Fix snapshot download since parameter to use closed/inclusive semantics

CHANGELOG.md

@@ -13,6 +13,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 - Added `make_cose_verifier_from_pem_cert()` and `make_cose_verifier_from_der_cert()` that accept certificates in a known format. The existing `make_cose_verifier_cert()` is renamed to `make_cose_verifier_any_cert()` (#7768).
 
+### Changed
+
+- The `since` query parameter on the `GET /node/snapshot` endpoint now uses closed (inclusive) semantics, consistent with the `since` parameter on `GET /node/ledger_chunk`. A request with `?since=N` will now return snapshots with index greater than or equal to `N`, rather than strictly greater than `N` (#7742).
+
 ## [7.0.0-dev13]
 
 [7.0.0-dev13]: https://github.com/microsoft/CCF/releases/tag/ccf-7.0.0-dev13

src/endpoints/authentication/cose_auth.cpp

@@ -224,7 +224,7 @@ namespace ccf
 
   MemberCOSESign1AuthnPolicy::MemberCOSESign1AuthnPolicy(
     std::optional<std::string> gov_msg_type_) :
-    gov_msg_type(std::move(gov_msg_type_)) {};
+    gov_msg_type(std::move(gov_msg_type_)){};
   MemberCOSESign1AuthnPolicy::~MemberCOSESign1AuthnPolicy() = default;
 
   std::unique_ptr<AuthnIdentity> MemberCOSESign1AuthnPolicy::authenticate(

src/js/extensions/ccf/historical.cpp

@@ -25,7 +25,7 @@ namespace ccf::js::extensions
     };
     std::unordered_map<ccf::SeqNo, HistoricalHandle> historical_handles;
 
-    Impl(ccf::historical::AbstractStateCache* hs) : historical_state(hs) {};
+    Impl(ccf::historical::AbstractStateCache* hs) : historical_state(hs){};
   };
 
   namespace

src/js/extensions/ccf/kv.cpp

@@ -24,7 +24,7 @@ namespace ccf::js::extensions
     std::optional<ccf::kv::CompactedVersionConflict>
       compacted_version_conflict = std::nullopt;
 
-    Impl(ccf::kv::Tx* t) : tx(t) {};
+    Impl(ccf::kv::Tx* t) : tx(t){};
   };
 
   namespace

src/node/node_state.h

@@ -3134,9 +3134,7 @@ namespace ccf
                   snapshot_evidence.version);
                 backup_snapshot_fetch_task =
                   std::make_shared<BackupSnapshotFetch>(
-                    config.snapshots,
-                    snapshot_evidence.version - 1 /* YIKES */,
-                    this);
+                    config.snapshots, snapshot_evidence.version, this);
                 ccf::tasks::add_task(backup_snapshot_fetch_task);
               }
             }

src/node/quote_endorsements_client.h

@@ -338,7 +338,7 @@ namespace ccf
       pal::snp::EndorsementEndpointsConfiguration config_,
       QuoteEndorsementsFetchedCallback cb) :
       config(std::move(config_)),
-      done_cb(std::move(cb)) {};
+      done_cb(std::move(cb)){};
 
     void fetch_endorsements()
     {

src/snapshots/filenames.h

@@ -187,7 +187,7 @@ namespace snapshots
         }
 
         const auto idx = get_snapshot_idx_from_file_name(file_name.string());
-        if (minimum_idx.has_value() && idx <= minimum_idx.value())
+        if (minimum_idx.has_value() && idx < minimum_idx.value())
         {
           LOG_DEBUG_FMT(
             "Ignoring snapshot file {} below minimum idx {}",

tests/e2e_operations.py

@@ -310,7 +310,7 @@ def do_request(http_verb, *args, **kwargs):
                 (1, location),
                 (snapshot_index // 2, location),
                 (snapshot_index - 1, location),
-                (snapshot_index, None),
+                (snapshot_index, location),
                 (snapshot_index + 1, None),
             ):
                 for method in ("GET", "HEAD"):