Merge pull request eclipse-thingweb#1284 from relu91/fix_1281

Console warning when no subprotocol is defined in http form
danielpeintner · May 15, 2024 · be63265 · be63265
2 parents 9d3858c + fc01e84
commit be63265
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/packages/binding-http/src/http-client-impl.ts b/packages/binding-http/src/http-client-impl.ts
@@ -158,7 +158,12 @@ export default class HttpClient implements ProtocolClient {
         complete?: () => void
     ): Promise<Subscription> {
         const defaultSubprotocol = "longpoll";
-        const subprotocol = form.subprotocol ?? defaultSubprotocol;
+        let subprotocol = form.subprotocol;
+
+        if (subprotocol == null) {
+            warn(`Subscribing to ${form.href} using long polling for form without subprotocol`);
+            subprotocol = defaultSubprotocol;
+        }
 
         let internalSubscription: InternalSubscription;
         if (subprotocol === defaultSubprotocol) {