Skip to content

Commit 6936a91

Browse files
Merge pull request #24 from fingerprintjs/rc
Rc
2 parents ffb9126 + a907c46 commit 6936a91

File tree

4 files changed

+18
-2
lines changed

4 files changed

+18
-2
lines changed

__mocks__/fastly:backend.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
export class Backend {
2+
static exists(backend) {
3+
console.log(`Check existence for ${backend}`)
4+
return true
5+
}
6+
}

src/handlers/handleIngressAPI.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { IntegrationEnv, isOpenClientResponseEnabled } from '../env'
1+
import { IntegrationEnv, isOpenClientResponseEnabled, isProxySecretSet } from '../env'
22
import {
33
addProxyIntegrationHeaders,
44
addTrafficMonitoringSearchParamsForVisitorIdRequest,
@@ -12,6 +12,9 @@ import { getIngressBackendByRegion } from '../utils/getIngressBackendByRegion'
1212
import { CacheOverride } from 'fastly:cache-override'
1313

1414
async function makeIngressRequest(receivedRequest: Request, env: IntegrationEnv) {
15+
if (!isProxySecretSet) {
16+
console.log("PROXY_SECRET is not set in the integration's Secret store, your integration is not working correctly.")
17+
}
1518
const url = new URL(receivedRequest.url)
1619
url.pathname = ''
1720
addTrafficMonitoringSearchParamsForVisitorIdRequest(url)

src/index.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,14 @@ export async function handleRequest(event: FetchEvent): Promise<Response> {
2929

3030
async function getEnvObject(): Promise<IntegrationEnv> {
3131
let configStore
32-
let secretStore
3332
try {
3433
configStore = getConfigStore()
34+
} catch (e) {
35+
console.error(e)
36+
}
37+
38+
let secretStore
39+
try {
3540
secretStore = getSecretStore()
3641
} catch (e) {
3742
console.error(e)

src/utils/createErrorResponse.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,10 +61,12 @@ export function createErrorResponseForIngress(request: Request, error: string |
6161
'Access-Control-Allow-Credentials': 'true',
6262
'content-type': 'application/json',
6363
}
64+
console.log(`Error occurred for ingress request, reason: ${reason}`)
6465
return new Response(JSON.stringify(responseBody), { status: 500, headers: responseHeaders })
6566
}
6667

6768
export function createFallbackErrorResponse(error: string | Error | unknown): Response {
6869
const responseBody = { error: errorToString(error) }
70+
console.log(`Error occurred, reason: ${responseBody}`)
6971
return new Response(JSON.stringify(responseBody), { status: 500, headers: { 'content-type': 'application/json' } })
7072
}

0 commit comments

Comments
 (0)