Skip to content

Commit

Permalink
fix: temporary fix for access log format to allow http gateway deploy (
Browse files Browse the repository at this point in the history
…#153)

Temporary fix for CI issue from
sst/sst#2587 as suggested in
sst/sst#2587 (comment)

The fix simply tweaks the `accessLog` format to not include the
`cognitoIdentityId` like fix from sst
https://github.com/serverless-stack/sst/pull/2602/files

In the meantime, hopefully sst team provides a fix for v1, or we need to
work on landing our update
#152 . I started by trying
to update, but after spending over 2 hours debugging random CI SegFaults
and other deployment issues decided to punt on this for now. v2 is quite
recent and maybe we should stay in v1 some more time given these
SegFault errors are likely not related to our code...
  • Loading branch information
vasco-santos authored Mar 7, 2023
1 parent 9c64395 commit cd1db3d
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions stacks/upload-api-stack.js
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,9 @@ export function UploadApiStack({ stack, app }) {
'GET /error': 'functions/get.error',
'GET /version': 'functions/get.version'
},
accessLog: {
format:'{"requestTime":"$context.requestTime","requestId":"$context.requestId","httpMethod":"$context.httpMethod","path":"$context.path","routeKey":"$context.routeKey","status":$context.status,"responseLatency":$context.responseLatency,"integrationRequestId":"$context.integration.requestId","integrationStatus":"$context.integration.status","integrationLatency":"$context.integration.latency","integrationServiceStatus":"$context.integration.integrationStatus","ip":"$context.identity.sourceIp","userAgent":"$context.identity.userAgent"}'
}
})

stack.addOutputs({
Expand Down

0 comments on commit cd1db3d

Please sign in to comment.