@travetto/web-http 7.0.0-rc.7 → 7.0.1

package/README.md

@@ -16,7 +16,7 @@ yarn add @travetto/web-http
 This module provides basic for running [http](https://nodejs.org/api/http.html). [https](https://nodejs.org/api/https.html)  and [http2](https://nodejs.org/api/http2.html) servers, along with support for tls key generation during development.
 
 ## Running a Server
-By default, the framework provides a default [@CliCommand](https://github.com/travetto/travetto/tree/main/module/cli/src/registry/decorator.ts#L85) for [WebHttpServer](https://github.com/travetto/travetto/tree/main/module/web-http/src/types.ts#L19) that will follow default behaviors, and spin up the server.
+By default, the framework provides a default [@CliCommand](https://github.com/travetto/travetto/tree/main/module/cli/src/registry/decorator.ts#L98) for [WebHttpServer](https://github.com/travetto/travetto/tree/main/module/web-http/src/types.ts#L19) that will follow default behaviors, and spin up the server.
 
 **Terminal: Standard application**
 ```bash
@@ -150,7 +150,7 @@ export class WebHttpConfig {
 ```
 
 ### Creating a Custom CLI Entry Point
-To customize a Web server, you may need to construct an entry point using the [@CliCommand](https://github.com/travetto/travetto/tree/main/module/cli/src/registry/decorator.ts#L85) decorator. This could look like:
+To customize a Web server, you may need to construct an entry point using the [@CliCommand](https://github.com/travetto/travetto/tree/main/module/cli/src/registry/decorator.ts#L98) decorator. This could look like:
 
 **Code: Application entry point for Web Applications**
 ```typescript

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/web-http",
-  "version": "7.0.0-rc.7",
+  "version": "7.0.1",
   "type": "module",
   "description": "Web HTTP Server Support",
   "keywords": [
@@ -27,11 +27,11 @@
     "directory": "module/web-http"
   },
   "dependencies": {
-    "@travetto/web": "^7.0.0-rc.6"
+    "@travetto/web": "^7.0.1"
   },
   "peerDependencies": {
-    "@travetto/cli": "^7.0.0-rc.5",
-    "@travetto/test": "^7.0.0-rc.5"
+    "@travetto/cli": "^7.0.1",
+    "@travetto/test": "^7.0.1"
   },
   "peerDependenciesMeta": {
     "@travetto/test": {

package/support/cli.web_http.ts

@@ -1,4 +1,4 @@
-import { Runtime, toConcrete, Util } from '@travetto/runtime';
+import { Runtime, toConcrete } from '@travetto/runtime';
 import { DependencyRegistryIndex } from '@travetto/di';
 import { CliCommand, CliCommandShape } from '@travetto/cli';
 import { NetUtil } from '@travetto/web';
@@ -9,7 +9,7 @@ import type { WebHttpServer } from '../src/types.ts';
 /**
  * Run a web server
  */
-@CliCommand({ runTarget: true, with: { debugIpc: true, restartForDev: true, module: true, env: true } })
+@CliCommand({ runTarget: true, with: { debugIpc: 'optional', restartOnChange: true, module: true, env: true } })
 export class WebHttpCommand implements CliCommandShape {
 
   /** Port to run on */
@@ -28,12 +28,16 @@ export class WebHttpCommand implements CliCommandShape {
     await Registry.init();
     const instance = await DependencyRegistryIndex.getInstance(toConcrete<WebHttpServer>());
 
-    if (this.killConflict) {
-      const handle = await Util.acquireWithRetry(() => instance.serve(), NetUtil.freePortOnConflict, 5);
-      return handle.complete;
-    } else {
+    try {
       const handle = await instance.serve();
       return handle.complete;
+    } catch (err) {
+      const result = this.killConflict ? await NetUtil.freePortOnConflict(err) : undefined;
+      if (result?.processId) {
+        console.warn('Killed process owning port', result);
+        process.exit(1);
+      }
+      throw err;
     }
   }
 }