Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add request headers into CloudWatch Logs #941

Closed
paynejd opened this issue Aug 24, 2021 · 3 comments
Closed

Add request headers into CloudWatch Logs #941

paynejd opened this issue Aug 24, 2021 · 3 comments
Assignees

Comments

@paynejd
Copy link
Member

paynejd commented Aug 24, 2021

CloudWatch is already enabled for OCL API and FHIR, but is capturing minimal data. We want to also capture request headers in preparation for using CloudWatch Insights

For staging, also change load balancer timeout to 2min and put a notification in place at 1min so that we can still track slow-running requests

@paynejd paynejd added this to the Sprint 17 milestone Aug 24, 2021
@paynejd paynejd removed this from the Sprint 17 milestone Sep 22, 2021
snyaggarwal added a commit to OpenConceptLab/oclapi2 that referenced this issue Nov 3, 2021
snyaggarwal added a commit to OpenConceptLab/oclapi2 that referenced this issue Nov 3, 2021
snyaggarwal added a commit to OpenConceptLab/oclapi2 that referenced this issue Nov 3, 2021
@jamlung-ri
Copy link
Contributor

@rkorytkowski Can you take a look at this and then address the Response Header follow-up ticket (#942)?

@snyaggarwal
Copy link
Contributor

Added request user in Response headers as X-OCL-REQUEST-USER

@snyaggarwal
Copy link
Contributor

Created #1116 to add response time in logs. There is no default Django way to do this, will have to write a custom middleware for this.
The rest of the tasks are done in this.
@rkorytkowski Please advise if we can close this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants