libstore: Make all StoreConfig::getReference implementations return store parameters

These stragglers have been accidentally left out when implementing the StoreConfig::getReference.
Also HttpBinaryCacheStore::getReference now returns the actual store parameters, not the cacheUri
parameters.

src/libstore-tests/http-binary-cache-store.cc

@@ -18,4 +18,20 @@ TEST(HttpBinaryCacheStore, constructConfigNoTrailingSlash)
     EXPECT_EQ(config.cacheUri.to_string(), "https://foo.bar.baz/a/b");
 }
 
+TEST(HttpBinaryCacheStore, constructConfigWithParams)
+{
+    StoreConfig::Params params{{"compression", "xz"}};
+    HttpBinaryCacheStoreConfig config{"https", "foo.bar.baz/a/b/", params};
+    EXPECT_EQ(config.cacheUri.to_string(), "https://foo.bar.baz/a/b");
+    EXPECT_EQ(config.getReference().params, params);
+}
+
+TEST(HttpBinaryCacheStore, constructConfigWithParamsAndUrlWithParams)
+{
+    StoreConfig::Params params{{"compression", "xz"}};
+    HttpBinaryCacheStoreConfig config{"https", "foo.bar.baz/a/b?some-param=some-value", params};
+    EXPECT_EQ(config.cacheUri.to_string(), "https://foo.bar.baz/a/b?some-param=some-value");
+    EXPECT_EQ(config.getReference().params, params);
+}
+
 } // namespace nix

src/libstore-tests/uds-remote-store.cc

@@ -22,4 +22,24 @@ TEST(UDSRemoteStore, constructConfig_to_string)
     EXPECT_EQ(config.getReference().to_string(), "daemon");
 }
 
+TEST(UDSRemoteStore, constructConfigWithParams)
+{
+    StoreConfig::Params params{{"max-connections", "1"}};
+    UDSRemoteStoreConfig config{"unix", "/tmp/socket", params};
+    auto storeReference = config.getReference();
+    EXPECT_EQ(storeReference.to_string(), "unix:///tmp/socket?max-connections=1");
+    EXPECT_EQ(storeReference.render(/*withParams=*/false), "unix:///tmp/socket");
+    EXPECT_EQ(storeReference.params, params);
+}
+
+TEST(UDSRemoteStore, constructConfigWithParamsNoPath)
+{
+    StoreConfig::Params params{{"max-connections", "1"}};
+    UDSRemoteStoreConfig config{"unix", "", params};
+    auto storeReference = config.getReference();
+    EXPECT_EQ(storeReference.to_string(), "daemon?max-connections=1");
+    EXPECT_EQ(storeReference.render(/*withParams=*/false), "daemon");
+    EXPECT_EQ(storeReference.params, params);
+}
+
 } // namespace nix

src/libstore/http-binary-cache-store.cc

@@ -39,7 +39,7 @@ StoreReference HttpBinaryCacheStoreConfig::getReference() const
                 .scheme = cacheUri.scheme,
                 .authority = cacheUri.renderAuthorityAndPath(),
             },
-        .params = cacheUri.query,
+        .params = getQueryParams(),
     };
 }
 

src/libstore/include/nix/store/dummy-store.hh

@@ -48,6 +48,7 @@ struct DummyStoreConfig : public std::enable_shared_from_this<DummyStoreConfig>,
                 StoreReference::Specified{
                     .scheme = *uriSchemes().begin(),
                 },
+            .params = getQueryParams(),
         };
     }
 };

src/libstore/s3-binary-cache-store.cc

@@ -262,6 +262,7 @@ StoreReference S3BinaryCacheStoreConfig::getReference() const
                 .scheme = *uriSchemes().begin(),
                 .authority = bucketName,
             },
+        .params = getQueryParams(),
     };
 }
 

src/libstore/uds-remote-store.cc

@@ -61,13 +61,17 @@ StoreReference UDSRemoteStoreConfig::getReference() const
      * to be more compatible with older versions of nix. Some tooling out there
      * tries hard to parse store references and it might not be able to handle "unix://". */
     if (path == settings.nixDaemonSocketFile)
-        return {.variant = StoreReference::Daemon{}};
+        return {
+            .variant = StoreReference::Daemon{},
+            .params = getQueryParams(),
+        };
     return {
         .variant =
             StoreReference::Specified{
                 .scheme = *uriSchemes().begin(),
                 .authority = path,
             },
+        .params = getQueryParams(),
     };
 }