Merge pull request #351 from whisperfish/auth-header

Use Auth header for ws login
whisperfish · Jan 16, 2025 · 19c0c78 · 19c0c78
2 parents dd43d93 + 67de052
commit 19c0c78
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/src/push_service/mod.rs b/src/push_service/mod.rs
@@ -219,20 +219,16 @@ impl PushService {
         let mut url = Endpoint::service(path).into_url(&self.cfg)?;
         url.set_scheme("wss").expect("valid https base url");
 
-        if let Some(credentials) = credentials {
-            url.query_pairs_mut()
-                .append_pair("login", &credentials.login())
-                .append_pair(
-                    "password",
-                    credentials.password.as_ref().expect("a password"),
-                );
-        }
-
         let mut builder = self.client.get(url);
         for (key, value) in additional_headers {
             builder = builder.header(*key, *value);
         }
 
+        if let Some(credentials) = credentials {
+            builder =
+                builder.basic_auth(credentials.login(), credentials.password);
+        }
+
         let ws = builder
             .upgrade()
             .send()