[Hawkular-commits] [hawkular/hawkular-metrics] ca18c7: [HWKMETRICS-589] Add REST API request logging

Branch: refs/heads/master Home: https://github.com/hawkular/hawkular-metrics Commit: ca18c772468a6fb25b2db53ab03400bfed149590 https://github.com/hawkular/hawkular-metrics/commit/ca18c772468a6fb25b2db... Author: John Sanda <jsanda(a)redhat.com&gt; Date: 2017-02-28 (Tue, 28 Feb 2017) Changed paths: M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/AvailabilityHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/GaugeHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/MetricHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/StringHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/TenantsHandler.java A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/Logged.java A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/RequestLoggingFilter.java M api/metrics-api-util/src/main/java/org/hawkular/metrics/api/jaxrs/config/ConfigurationKey.java Log Message: ----------- [HWKMETRICS-589] Add REST API request logging There are two properties that control request logging one of which is for writes. I felt it necessary to have a separate property for logging writes because it could result in lots of extra logging. The output looks like: [[0m[[0m11:34:31,643 INFO [org.hawkular.metrics.api.jaxrs.util.RequestLoggingFilter] (default task-49) REST API request: -------------------------------------- path: /metrics segments: [metrics] method: GET query parameters: {type=[availability]} headers: [Accept=application/json, application/javascript, text/javascript,Connection=Keep-Alive,Hawkular-Tenant=T9a116f18-28cf-41b3-8ff8-c9752ac60e26232,Host=127.0.0.1:55977] Commit: 6f1dfb819c17fd219d5033cd8aefa9765d2d8cbe https://github.com/hawkular/hawkular-metrics/commit/6f1dfb819c17fd219d503... Author: John Sanda <jsanda(a)redhat.com&gt; Date: 2017-02-28 (Tue, 28 Feb 2017) Changed paths: M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/TenantsHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/Logged.java Log Message: ----------- [HWKMETRICS-589] javadocs and license header Commit: 644355e03f85ab93b72daa485bb15229ce73a6c1 https://github.com/hawkular/hawkular-metrics/commit/644355e03f85ab93b72da... Author: John Sanda <jsanda(a)redhat.com&gt; Date: 2017-02-28 (Tue, 28 Feb 2017) Changed paths: M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/Logged.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/RequestLoggingFilter.java Log Message: ----------- [HWKMETRICS-589] license header Commit: 290a7ec77e0888e8a0c6a5838311516c16e0a20e https://github.com/hawkular/hawkular-metrics/commit/290a7ec77e0888e8a0c6a... Author: John Sanda <jsanda(a)redhat.com&gt; Date: 2017-02-28 (Tue, 28 Feb 2017) Changed paths: A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/filter/RequestLoggingFilter.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/Logged.java R api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/RequestLoggingFilter.java M api/metrics-api-util/src/main/java/org/hawkular/metrics/api/jaxrs/config/ConfigurationKey.java Log Message: ----------- [HWKMETRICS-589] do not enable logging automatically and only log tenant header Either or both the logging config properties need to be set in order to turn on logging. We are only logging the tenant header to avoid logging any security sensitive info. Commit: d45e7c5972ecb71243dc97fb86cc1b06dbdf5311 https://github.com/hawkular/hawkular-metrics/commit/d45e7c5972ecb71243dc9... Author: Stefan Negrea <stefannegrea(a)users.noreply.github.com&gt; Date: 2017-02-28 (Tue, 28 Feb 2017) Changed paths: A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/filter/RequestLoggingFilter.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/AvailabilityHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/CounterHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/GaugeHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/MetricHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/StringHandler.java M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/handler/TenantsHandler.java A api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/util/Logged.java M api/metrics-api-util/src/main/java/org/hawkular/metrics/api/jaxrs/config/ConfigurationKey.java Log Message: ----------- Merge pull request #772 from jsanda/hwkmetrics-589 [HWKMETRICS-589] Add REST API request logging Compare: https://github.com/hawkular/hawkular-metrics/compare/5caeb03c3cb4...d45e7...