Merge pull request #160 from ing-bank/fix/username-in-metrics

Full user name in Q metrics

Author: arempter

.github/workflows/ci-pr.yml

@@ -5,7 +5,7 @@ on:
       - master
 jobs:
   it-test:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-18.04
     steps:
       - name: Checkout
         uses: actions/checkout@v1
@@ -18,7 +18,7 @@ jobs:
       - name: "Starting dependent containers for testing"
         run: docker-compose up -d keycloak ceph postgres-server mariadb ranger-admin rokku-sts cassandra
       - name: "Install aws cli"
-        run: pip install --user awscli
+        run: pip install --user awscli==1.18.222
       - name: "Compile the project while containers are starting up"
         run: sbt clean compile
       - name: "Wait for containers to be up and running"

src/main/scala/com/ing/wbaa/rokku/proxy/queue/MemoryUserRequestQueue.scala

@@ -82,6 +82,6 @@ trait MemoryUserRequestQueue extends UserRequestQueue {
   }
 
   private def metricName(user: User): String = {
-    MetricsFactory.REQUEST_QUEUE_OCCUPIED_BY_USER.replace(MetricsFactory.REQUEST_USER, s"${user.userName.value.head.toString}..")
+    MetricsFactory.REQUEST_QUEUE_OCCUPIED_BY_USER.replace(MetricsFactory.REQUEST_USER, user.userName.value)
   }
 }