Add platform info to access logging only in Backsplash #5638
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
This PR:
UserWithPlatform
which hasplatformId
andplatformName
in addition to user fieldsUserWithPlatform
with response headersAccessLoggingMiddleware
so that it reads from the above updated header info, adds platform name and ID to the access logs, and delete the added headers before returning the responseChecklist
New tables and queries have appropriate indices addedAny content changes are properly templated usingBUILDCONFIG.APP_NAME
Any new SQL strings have testsAny new endpoints have scope validation and are included in the integration test csvTesting Instructions
test/asu/add-platId-to-access-logging
branch was pushed to staging, so the current staging backsplash should already ship platform information along with existing access logs to Cloudwatchtest
branch again to test it on staging and Cloudwatch, since this should be the easiest way to testlogStagingBacksplash
log group, set the timeframe as the last 1 hour, and run the following query. It should give you some results that contain platform info in the access logsCloses https://github.com/azavea/raster-foundry-platform/issues/1370