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

Disables response logging as a default #42353

Merged
merged 3 commits into from Aug 12, 2019

Conversation

tylersmalley
Copy link
Contributor

Previously, if any of the following were true, we would log HTTP responses.

  • logging.json was set
  • logging.dest was set
  • TTY was detected

The logging format should not dictate what is logged

Release Note: Responses are never logged by default. Previously, responses would be logged if either logging.json was true, logging.dest was specified, or a TTY was detected. To restore the previous behavior set logging.events.response=*` in kibana.yml.

Previously, if any of the following were true, we would log HTTP responses.
- `logging.json` was set
- `logging.dest` was set
- `TTY` was detected

The logging format should not dictate what is logged

Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@tylersmalley
Copy link
Contributor Author

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@tylersmalley tylersmalley merged commit 372b79e into elastic:master Aug 12, 2019
chrisronline pushed a commit to chrisronline/kibana that referenced this pull request Aug 15, 2019
Previously, if any of the following were true, we would log HTTP responses.
- `logging.json` was set
- `logging.dest` was set
- `TTY` was detected

The logging format should not dictate what is logged

Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
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

3 participants