refactor: add counter to differentiate instead of infinite underscores

Author: danielpeintner

packages/binding-coap/src/coap-server.ts

@@ -160,8 +160,14 @@ export default class CoapServer implements ProtocolServer {
     private createThingUrlPath(thing: ExposedThing) {
         let urlPath = slugify(thing.title, { lower: true });
 
-        while (this.things.has(urlPath)) {
-            urlPath += "_";
+        // avoid URL clashes
+        if (this.things.has(urlPath)) {
+            let uniqueUrlPath;
+            let nameClashCnt = 2;
+            do {
+                uniqueUrlPath = urlPath + "_" + nameClashCnt++;
+            } while (this.things.has(uniqueUrlPath));
+            urlPath = uniqueUrlPath;
         }
 
         return urlPath;

packages/binding-http/src/http-server.ts

@@ -269,8 +269,14 @@ export default class HttpServer implements ProtocolServer {
     public async expose(thing: ExposedThing, tdTemplate: WoT.ExposedThingInit = {}): Promise<void> {
         let urlPath = slugify(thing.title, { lower: true });
 
-        while (this.things.has(urlPath)) {
-            urlPath += "_";
+        // avoid URL clashes
+        if (this.things.has(urlPath)) {
+            let uniqueUrlPath;
+            let nameClashCnt = 2;
+            do {
+                uniqueUrlPath = urlPath + "_" + nameClashCnt++;
+            } while (this.things.has(uniqueUrlPath));
+            urlPath = uniqueUrlPath;
         }
 
         if (this.getPort() !== -1) {

packages/binding-websockets/src/ws-server.ts

@@ -158,8 +158,14 @@ export default class WebSocketServer implements ProtocolServer {
     public expose(thing: ExposedThing): Promise<void> {
         let urlPath = slugify(thing.title, { lower: true });
 
-        while (this.thingNames.has(urlPath)) {
-            urlPath += "_";
+        // avoid name clashes
+        if (this.thingNames.has(urlPath)) {
+            let uniqueUrlPath;
+            let nameClashCnt = 2;
+            do {
+                uniqueUrlPath = urlPath + "_" + nameClashCnt++;
+            } while (this.thingNames.has(uniqueUrlPath));
+            urlPath = uniqueUrlPath;
         }
 
         if (this.getPort() !== -1) {