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

Fix #1498: enable access/request log by default #1503

Merged
merged 3 commits into from
Oct 7, 2024

Conversation

tatu-at-datastax
Copy link
Contributor

@tatu-at-datastax tatu-at-datastax commented Oct 7, 2024

What this PR does:

Enables Quarkus HTTP access/request log by default

Resulting entry as of now looks like:

INFO  [vert.x-eventloop-thread-1] 2024-10-07 09:40:07,330 JBossLoggingAccessLogReceiver.java:44 - 127.0.0.1 - [07/Oct/2024:09:40:07 -0700] "POST /v1/example HTTP/1.1" 200 29

Which issue(s) this PR fixes:
Fixes #1498

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

@tatu-at-datastax tatu-at-datastax self-assigned this Oct 7, 2024
@tatu-at-datastax tatu-at-datastax requested a review from a team as a code owner October 7, 2024 16:05
@@ -115,6 +116,8 @@ quarkus:
level: ERROR
'com.datastax.oss.driver':
level: WARN
'io.quarkus.http.access-log':
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is important: with default level of WARN request/access log would not be included without.

@tatu-at-datastax tatu-at-datastax merged commit 5b6d6af into main Oct 7, 2024
3 checks passed
@tatu-at-datastax tatu-at-datastax deleted the tatu/1498-enable-request-log branch October 7, 2024 17:06
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.

Enable HTTP request log (access log) in Production
2 participants