Merge pull request #20 from fingerprintjs/rc

Release v0.2.1-rc.1

Author: Orkuncakilkaya

.releaserc.json

@@ -48,8 +48,8 @@
       {
         "assets":[
           {
-            "path":"pkg/package.tar.gz",
-            "label":"fingerprint-proxy-integration.tar.gz"
+            "path":"pkg/fingerprint-fastly-compute-proxy-integration.tar.gz",
+            "label":"fingerprint-fastly-compute-proxy-integration.tar.gz"
           }
         ]
       }

CHANGELOG.md

@@ -1,3 +1,11 @@
+## [0.2.1-rc.1](https://github.com/fingerprintjs/fingerprint-pro-fastly-compute-proxy-integration/compare/v0.2.0...v0.2.1-rc.1) (2024-11-14)
+
+
+### Bug Fixes
+
+* change release artifact name ([a3d44da](https://github.com/fingerprintjs/fingerprint-pro-fastly-compute-proxy-integration/commit/a3d44da4628930f73f5d0d87d3f64ee0d20b8ca4))
+* move plugins to the next tick of event loop ([6a3d554](https://github.com/fingerprintjs/fingerprint-pro-fastly-compute-proxy-integration/commit/6a3d5547ad9c928b436431c5b4212abf473f4c1b))
+
 ## [0.2.0](https://github.com/fingerprintjs/fingerprint-pro-fastly-compute-proxy-integration/compare/v0.1.0...v0.2.0) (2024-11-11)
 
 

README.md

@@ -84,7 +84,7 @@ To use a custom name prefix for both stores, use the `STORE_NAME_PREFIX` environ
 STORE_NAME_PREFIX=CustomName pnpm run build
 ```
 
-Your custom built package in `pkg/package.tar.gz` will use your custom prefix in store names like:
+Your custom built package in `pkg/fingerprint-fastly-compute-proxy-integration.tar.gz` will use your custom prefix in store names like:
 
 * `CustomName_Config_Store_<SERVICE_ID>`
 * `CustomName_Secret_Store_<SERVICE_ID>`

contributing.md

@@ -10,12 +10,12 @@
 
 We prefer using [pnpm](https://pnpm.io/) for managing dependencies and running scripts.
 
-For proposing changes, use the standard [pull request approach](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request). It’s recommended to discuss fixes or new features in the Issues first.
-
 - The `main` branch is locked for direct pushes.
 - The `rc` branch is used for active development of new features.
 - Releases are created from the `main` branch.
 
+For proposing changes, use the standard [pull request approach](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request). Please ensure that your pull request is directed to the `rc` branch, as this is our active development branch for new features. You can create a PR to the `rc` branch from your own branch [here](https://github.com/fingerprintjs/fingerprint-pro-fastly-compute-proxy-integration/compare/rc...). It’s recommended to discuss fixes or new features in the Issues first.
+
 ### How to Build
 
 1. After cloning the repository, run `pnpm install` to install dependencies.

fastly.toml

@@ -1,21 +1,14 @@
 # This file describes a Fastly Compute package. To learn more visit:
 # https://developer.fastly.com/reference/fastly-toml/
 
-authors = []
-cloned_from = "https://github.com/fastly/compute-starter-kit-typescript"
+authors = ["[email protected]"]
+cloned_from = "https://github.com/fingerprintjs/fingerprint-pro-fastly-compute-proxy-integration"
 description = "Fingerprint Proxy Fastly Compute Package"
 language = "javascript"
 manifest_version = 3
-name = "fingerprint-pro-fastly-proxy-integration"
+name = "fingerprint-fastly-compute-proxy-integration"
 service_id = ""
 
 [scripts]
   build = "pnpm build"
   post_init = "pnpm install"
-
-#[local_server]
-#  [local_server.backends]
-#    [local_server.backends.fpcdn]
-#      url = "http://localhost:1234"
-#    [local_server.backends.fpjs]
-#      url = "http://localhost:1234"

package.json

@@ -1,6 +1,6 @@
 {
   "name": "fingerprint-pro-fastly-compute-proxy-integration",
-  "version": "0.2.0",
+  "version": "0.2.1-rc.1",
   "engines": {
     "node": ">=20"
   },
@@ -41,7 +41,7 @@
   "scripts": {
     "prebuild": "ts-node build.ts",
     "assemble": "js-compute-runtime build/index.js bin/main.wasm",
-    "pack": "fastly compute pack --wasm-binary=./bin/main.wasm",
+    "pack": "fastly compute pack --wasm-binary=./bin/main.wasm && mv pkg/package.tar.gz pkg/fingerprint-fastly-compute-proxy-integration.tar.gz",
     "build": "pnpm run prebuild && pnpm run assemble && pnpm run pack",
     "deploy": "fastly compute publish",
     "dev": "fastly compute serve",

scripts/api/deployPackage.ts

@@ -5,6 +5,6 @@ export async function deployPackage(service_id: string, versionId: number) {
   return createClient('package').putPackage({
     version_id: versionId,
     service_id,
-    _package: fs.createReadStream(__dirname + '/../../pkg/fingerprint-pro-fastly-proxy-integration.tar.gz'),
+    _package: fs.createReadStream(__dirname + '/../../pkg/fingerprint-fastly-compute-proxy-integration.tar.gz'),
   })
 }

src/handlers/handleIngressAPI.ts

@@ -30,17 +30,19 @@ async function makeIngressRequest(receivedRequest: Request, env: IntegrationEnv)
 
   if (!isOpenClientResponseEnabled(env)) {
     console.log(
-      "Open client response plugings are disabled. Set OPEN_CLIENT_RESPONSE_PLUGINS_ENABLED to `true` in your proxy integration's Config store to enable them."
+      "Open client response plugins are disabled. Set OPEN_CLIENT_RESPONSE_PLUGINS_ENABLED to `true` in your proxy integration's Config store to enable them."
     )
     return response
   }
 
   console.log('Plugin system for Open Client Response is enabled')
   if (response.status >= 200 && response.status < 300) {
     const responseBody = await response.text()
-    processOpenClientResponse(responseBody, response, env).catch((e) =>
-      console.error('Processing open client response failed: ', e)
-    )
+    Promise.resolve().then(() => {
+      processOpenClientResponse(responseBody, response, env).catch((e) =>
+        console.error('Processing open client response failed: ', e)
+      )
+    })
     return cloneFastlyResponse(responseBody, response)
   }
 

src/handlers/handleStatusPage.ts

@@ -110,7 +110,7 @@ function createEnvVarsInformationElement(env: IntegrationEnv): string {
     result += `
       <span>
       ${configuration.isSet ? '✅' : '⚠️'} <strong>${configuration.label} </strong> (${configuration.required ? 'REQUIRED' : 'OPTIONAL'}) is${!configuration.isSet ? ' not ' : ' '}set
-      ${!configuration.isSet && configuration.message ? `<span>${configuration.message}</span>` : ''}
+      ${configuration.required && !configuration.isSet && configuration.message ? `<span>${configuration.message}</span>` : ''}
       </span>
       `
   }