prop250: Silence a logging message.

LOG_NOTICE is a bit too much for that one.

1 files changed, 10 insertions, 2 deletions

diff --git a/src/or/shared_random.c b/src/or/shared_random.c
index 7da6daf62f..599273f066 100644
--- a/src/or/shared_random.c
+++ b/src/or/shared_random.c
@@ -438,7 +438,7 @@ generate_srv(const char *hashed_reveals, uint8_t reveal_num,
     /* Debugging. */
     char srv_hash_encoded[SR_SRV_VALUE_BASE64_LEN + 1];
     sr_srv_encode(srv_hash_encoded, sizeof(srv_hash_encoded), srv);
-    log_debug(LD_DIR, "SR: Generated SRV: %s", srv_hash_encoded);
+    log_info(LD_DIR, "SR: Generated SRV: %s", srv_hash_encoded);
   }
   return srv;
 }
@@ -617,7 +617,15 @@ should_keep_commit(const sr_commit_t *commit, const char *voter_key,
   case SR_PHASE_COMMIT:
     /* Already having a commit for an authority so ignore this one. */
     if (saved_commit) {
-      log_debug(LD_DIR, "SR: Ignoring known commit during COMMIT phase.");
+      /*  Receiving known commits should happen naturally since commit phase
+          lasts multiple rounds. However if the commitment value changes
+          during commit phase, it might be a bug so log more loudly. */
+      if (!commitments_are_the_same(commit, saved_commit)) {
+        log_info(LD_DIR, "SR: Received altered commit from %s in commit phase.",
+                 sr_commit_get_rsa_fpr(commit));
+      } else {
+        log_debug(LD_DIR, "SR: Ignoring known commit during commit phase.");
+      }
       goto ignore;
     }