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

feat(config): add captureHeaders config #788

Merged
merged 4 commits into from Jan 22, 2019
Merged

Conversation

Qard
Copy link
Contributor

@Qard Qard commented Jan 15, 2019

Still needs more tests and documentation.

Fixes #739

Checklist

  • Implement code
  • Add tests
  • Update documentation

lib/parsers.js Show resolved Hide resolved
lib/parsers.js Show resolved Hide resolved
test/parsers.js Outdated Show resolved Hide resolved
@Qard
Copy link
Contributor Author

Qard commented Jan 18, 2019

Added docs.

docs/configuration.asciidoc Outdated Show resolved Hide resolved
Co-Authored-By: Qard <admin@stephenbelanger.com>
watson
watson previously approved these changes Jan 18, 2019
@watson watson merged commit 86a4237 into elastic:master Jan 22, 2019
@watson
Copy link
Member

watson commented Jan 22, 2019

@Qard I just realized that this was marked in-progress and not in-review. I thought it was finished though?

@Qard
Copy link
Contributor Author

Qard commented Jan 22, 2019

Sorry, I just forgot to move it to in-review. It's all good. 👍

@Qard Qard deleted the capture-headers branch January 22, 2019 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants