From ec0a9c7b802dbf996460c1f09f67772b3e645efb Mon Sep 17 00:00:00 2001 From: Maxime Dor Date: Wed, 13 Sep 2017 16:20:46 +0200 Subject: [PATCH] log user agent for lookup requests --- .../io/kamax/mxisd/controller/v1/MappingController.groovy | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/main/groovy/io/kamax/mxisd/controller/v1/MappingController.groovy b/src/main/groovy/io/kamax/mxisd/controller/v1/MappingController.groovy index fdd9ef3..faefd0f 100644 --- a/src/main/groovy/io/kamax/mxisd/controller/v1/MappingController.groovy +++ b/src/main/groovy/io/kamax/mxisd/controller/v1/MappingController.groovy @@ -65,6 +65,8 @@ class MappingController { if (lookupReq.isRecursive()) { lookupReq.setRecurseHosts(Arrays.asList(xff.split(","))) } + + lookupReq.setUserAgent(req.getHeader("USER-AGENT")) } @RequestMapping(value = "/lookup", method = GET) @@ -74,7 +76,7 @@ class MappingController { lookupRequest.setType(medium) lookupRequest.setThreePid(address) - log.info("Got request from {} - Is recursive? {}", lookupRequest.getRequester(), lookupRequest.isRecursive()) + log.info("Got single lookup request from {} with client {} - Is recursive? {}", lookupRequest.getRequester(), lookupRequest.getUserAgent(), lookupRequest.isRecursive()) Optional lookupOpt = strategy.find(lookupRequest) if (!lookupOpt.isPresent()) { @@ -99,7 +101,7 @@ class MappingController { String bulkLookup(HttpServletRequest request) { BulkLookupRequest lookupRequest = new BulkLookupRequest() setRequesterInfo(lookupRequest, request) - log.info("Got request from {} - Is recursive? {}", lookupRequest.getRequester(), lookupRequest.isRecursive()) + log.info("Got single lookup request from {} with client {} - Is recursive? {}", lookupRequest.getRequester(), lookupRequest.getUserAgent(), lookupRequest.isRecursive()) ClientBulkLookupRequest input = (ClientBulkLookupRequest) json.parseText(request.getInputStream().getText()) List mappings = new ArrayList<>()