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 support for http auth #17

Closed
wants to merge 3 commits into from
Closed

Conversation

myoung34
Copy link

@myoung34 myoung34 commented Sep 25, 2017

Resolves #16

Redaction necesary, or user/pass shows up in log. After redacting:

/aws/lambda/cloudwatch_logs_to_elasticsearch 2017/09/26/[$LATEST]cde5dc28309c4228b2b265c1cea26e9f 2017-09-26T18:12:08.322Z      35574970-a2e6-11e7-a601-6df19095e394    Running 5 handlers with _config: { elasticsearch:
   { host: 'https://es.redact.com:9200',
     httpAuth: '*****',
     index: 'logstash-2017-09-26',
     type: 'test' },
  data: 'snip',
  type: 'CloudWatch',
  processors: [ 'formatCloudwatchLogs', 'shipElasticsearch' ] }

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling e82dae6 on myoung34:master into 729f9ba on arithmetric:master.

@myoung34 myoung34 force-pushed the master branch 4 times, most recently from 371f06a to d2df96b Compare September 25, 2017 20:07
@coveralls
Copy link

Coverage Status

Coverage decreased (-89.8%) to 10.169% when pulling 305d38f on myoung34:master into 729f9ba on arithmetric:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 371f06a on myoung34:master into 729f9ba on arithmetric:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d2df96b on myoung34:master into 729f9ba on arithmetric:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d2df96b on myoung34:master into 729f9ba on arithmetric:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 0566dfd on myoung34:master into 729f9ba on arithmetric:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 966fcc2 on myoung34:master into 729f9ba on arithmetric:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d11b3d0 on myoung34:master into 729f9ba on arithmetric:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5c028d1 on myoung34:master into 729f9ba on arithmetric:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-7.4%) to 92.642% when pulling cbbf374 on myoung34:master into 729f9ba on arithmetric:master.

@myoung34
Copy link
Author

This got noisy, opening new PR

@myoung34 myoung34 closed this Sep 29, 2017
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