commit 105210beb0c0de5ed3ce5e89ef5f77f444a7d834
author Matteo Scandolo <teo@opennetworking.org> Thu Dec 10 10:32:39 2020 -0800
committer Matteo Scandolo <teo@opennetworking.org> Thu Dec 10 10:41:30 2020 -0800
tree 5f3c7c9f5543bb33e61590b109b6926b9dce2b52
parent eca0e8d2e3aade1067257793226951a23954460b

[VOL-3591] Using getReadableIdentifier throughout the app to cleanup logs

Change-Id: Id977851e8e9a5a02b76d8ee43f5356d4185ea803

app/src/main/java/org/opencord/aaa/impl/IdentifierManager.java

diff --git a/app/src/main/java/org/opencord/aaa/impl/IdentifierManager.java b/app/src/main/java/org/opencord/aaa/impl/IdentifierManager.java
index 6b4bffb..911f577 100644
--- a/app/src/main/java/org/opencord/aaa/impl/IdentifierManager.java
+++ b/app/src/main/java/org/opencord/aaa/impl/IdentifierManager.java
@@ -137,23 +137,23 @@
      */
     public synchronized void releaseIdentifier(RequestIdentifier id) {
         if (log.isTraceEnabled()) {
-            log.trace("Releasing identifier {}", id.identifier() & 0xff);
+            log.trace("Releasing identifier {}", id.getReadableIdentifier());
         }
 
         Pair<String, Long> session = idToSession.remove(id);
         if (session == null) {
             if (log.isTraceEnabled()) {
-                log.trace("Unable to released identifier {} for session null", id.identifier() & 0xff);
+                log.trace("Unable to released identifier {} for session null", id.getReadableIdentifier());
             }
             // this id wasn't mapped to a session so is still free
             return;
         }
 
         // add id number back to set of free ids
-        freeIdNumbers.add(id.identifier() & 0xff);
+        freeIdNumbers.add(id.getReadableIdentifier());
 
         if (log.isTraceEnabled()) {
-            log.trace("Released identifier {} for session {}", id.identifier() & 0xff, session.getKey());
+            log.trace("Released identifier {} for session {}", id.getReadableIdentifier(), session.getKey());
         }
     }
 
@@ -191,7 +191,7 @@
             if (diff >= timeout) {
                 if (log.isTraceEnabled()) {
                     log.trace("Identifier {} for session {} has exceeded timeout {}, releasing",
-                            id.identifier() & 0xff, info.getKey(), timeout);
+                            id.getReadableIdentifier(), info.getKey(), timeout);
                 }
                 releaseIdentifier(id);
             }