fix: drop finally block and enable keepalive

Signed-off-by: Sam Gammon <[email protected]>

Author: sgammon

dist/cleanup/index.js

@@ -101,7 +101,8 @@ const userAgent = userAgentSegments.join(' ')
 
 export const httpClient = new http.HttpClient(userAgent, [], {
   allowRetries: true,
-  maxRetries: 3
+  maxRetries: 3,
+  keepAlive: true
 })
 
 /**

dist/cleanup/index.js.map

@@ -143,7 +143,10 @@ async function transmitEvent(event: ActionEvent<any>): Promise<void> {
       encoded,
       headers
     )
-    if (res.message.statusCode !== 200 && res.message.statusCode !== 202) {
+    if (
+      res.message.statusCode !== http.HttpCodes.OK &&
+      res.message.statusCode !== 202
+    ) {
       core.debug(`Event transmission failed: ${res.message.statusMessage}`)
     } else {
       core.debug(`Event transmission completed (ID: ${event.uuid})`)
@@ -216,5 +219,5 @@ export async function error(err: Error | unknown, fatal = true): Promise<void> {
  * Exit hook for making sure Sentry finishes reporting.
  */
 export async function onExit(): Promise<void> {
-  // Nothing to do.
+  await Sentry.close(2000)
 }

dist/index.js

@@ -579,12 +579,11 @@ export async function entry(options?: Partial<Options>): Promise<void> {
 
   try {
     await install(options || {}, true)
+    await onExit()
   } catch (err) {
     core.warning(
       'Buildless failed to install; this build may not be accelerated. Please see CI logs for more information.'
     )
-  } finally {
-    await onExit()
   }
 }
 
@@ -599,11 +598,10 @@ export async function cleanup(options?: Partial<Options>): Promise<void> {
   try {
     await postExecute(options)
     core.info(`Thanks for using Buildless. 🎉`)
+    await onExit()
   } catch (err) {
     core.notice(
       'Cleanup stage for the Buildless action failed. Please see CI logs for more information.'
     )
-  } finally {
-    await onExit()
   }
 }