Merge pull request #216 from mpolgar/slightly_extra_logging

Add logging for changes made from /push endpoint

Author: belvedere-trading-user

core/src/main/java/com/confighub/core/store/Store.java

@@ -1981,6 +1981,8 @@ public void savePropertyKey( final String appIdentity,
     {
         validateWriteAccess( repository, token, propertyKey );
         saveOrUpdateAuditedViaAPI( appIdentity, repository, propertyKey, changeComment );
+
+        log.info( "Updated property key for repository [%s] with key [%s]", repository.getName(), propertyKey.getKey() );
     }
 
 
@@ -1993,6 +1995,12 @@ public void saveProperty( final String appIdentity,
     {
         validateWriteAccess( repository, token, property );
         saveOrUpdateAuditedViaAPI( appIdentity, repository, property, changeComment );
+
+        log.info( "Updated property value > %s for repository [%s] and key [%s]: %s",
+                ContextParser.contextToString(property.getContext(), repository),
+                repository.getName(),
+                property.getPropertyKey().getKey(),
+                property.getValue() != null ? property.getValue().replace("\n", "\\n") : "" );
     }
 
 
@@ -2056,7 +2064,7 @@ public Property createProperty( final String key,
                 propertyKey.setSecurityProfile( sp, spPassword );
             }
 
-            log.info( "Added property for repository [%s] with key [%s]", repository.getName(), key );
+            log.info( "Added property key for repository [%s] with key [%s]", repository.getName(), key );
         }
 
         property.setPropertyKey( propertyKey );
@@ -2346,7 +2354,7 @@ public boolean deleteKeyAndProperties( final UserAccount user,
         }
 
         deleteAudited( user, repository, key );
-        log.info( "Deleted property for repository [%s] with key [%s]", repository.getName(), keyString );
+        log.info( "Deleted property key for repository [%s] with key [%s]", repository.getName(), keyString );
         return true;
     }
 
@@ -2392,7 +2400,7 @@ public boolean deleteKeyAndProperties( final String appIdentity,
         }
 
         deleteAuditedViaAPI( appIdentity, repository, key, changeComment );
-        log.info( "Deleted property for repository [%s] with key [%s]", repository.getName(), keyString );
+        log.info( "Deleted property key for repository [%s] with key [%s]", repository.getName(), keyString );
         return true;
     }
 
@@ -2455,7 +2463,7 @@ public boolean deleteProperty( final UserAccount user,
                                                                                                        .size() == 0 ) )
         {
             ret &= deleteAudited( user, repository, propertyKey );
-            log.info( "Deleted property for repository [%s] with key [%s]", repository.getName(), key );
+            log.info( "Deleted property key for repository [%s] with key [%s]", repository.getName(), key );
         }
 
         return ret;
@@ -2505,7 +2513,7 @@ public boolean deleteProperty( final String appIdentity,
                                                                                                        .size() == 0 ) )
         {
             ret &= deleteAuditedViaAPI( appIdentity, repository, propertyKey, changeComment );
-            log.info( "Deleted property for repository [%s] with key [%s]", repository.getName(), key );
+            log.info( "Deleted property key for repository [%s] with key [%s]", repository.getName(), key );
         }
 
         return ret;
@@ -2784,7 +2792,7 @@ public Pair<PropertyKey, KeyUpdateStatus> updatePropertyKey( final Repository re
             validateWriteAccess( repository, user, propertyKey );
             saveOrUpdateAudited( user, repository, propertyKey, changeComment );
 
-            log.info( "Added property for repository [%s] with key [%s]", repository.getName(), newKeyString );
+            log.info( "Added property key for repository [%s] with key [%s]", repository.getName(), newKeyString );
 
             return new Pair<>( propertyKey, KeyUpdateStatus.UPDATE );
         }
@@ -3276,7 +3284,7 @@ else if ( toSave.isSecure() )
             deleteAudited( user, repository, toDelete, changeComment );
         }
 
-        log.info( "Updated property for repository [%s] with key [%s]", repository.getName(), newKeyString );
+        log.info( "Updated property key for repository [%s] with key [%s]", repository.getName(), newKeyString );
         return new Pair<>( toSave, status );
     }
 

pom.xml

@@ -16,6 +16,6 @@
         <module>rest</module>
     </modules>
     <properties>
-        <revision>1.10.5</revision>
+        <revision>1.10.6</revision>
     </properties>
 </project>