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

Take a stab at logging query to CloudWatch #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

reefdog
Copy link
Contributor

@reefdog reefdog commented Jul 29, 2019

This is my attempt at logging the request.query to CloudWatch.

I’m not sure this is the correct way, but intended to just build and upyoload, but I can’t get Lambda to take my built JAR so I’m committing this possibly wrong code instead. Hopefully @joeltg will have a better time uploading this than me, or will figure out why it's wrong.

The end result, hopefully, is being able to see the request.query show up in this list of log events at CloudWatch.

Affects #3

This is my attempt at logging the `request.query` to CloudWatch.

I’m not sure this is the correct way, and intended to just build, YOLO,
and upload, but I can’t get Lambda to take my built JAR so I’m
committing this possibly wrong code instead.

Affects #3
@reefdog reefdog requested a review from joeltg July 29, 2019 23:37
@reefdog reefdog removed the request for review from joeltg July 30, 2019 06:05
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

Successfully merging this pull request may close these issues.

2 participants