crc: 404 for unknown monitoring request, only increment dkg after sub…

…mission

stacks-signer/src/monitoring/server.rs

@@ -119,9 +119,24 @@ impl MonitoringServer {
                 continue;
             }
 
-            // unknown request, return 200 ok
+            if request.url() == "/info" {
+                request
+                    .respond(HttpResponse::from_string(self.get_info_response()))
+                    .expect("Failed to respond to request");
+                continue;
+            }
+
+            // return 200 OK for "/"
+            if request.url() == "/" {
+                request
+                    .respond(HttpResponse::from_string("OK"))
+                    .expect("Failed to respond to request");
+                continue;
+            }
+
+            // unknown request, return 404
             request
-                .respond(HttpResponse::from_string(self.get_info_response()))
+                .respond(HttpResponse::from_string("Not Found").with_status_code(404))
                 .expect("Failed to respond to request");
         }
     }

stacks-signer/src/signer.rs

@@ -1212,11 +1212,11 @@ impl Signer {
             debug!("{self}: Received a DKG result, but are in an unsupported epoch. Do not broadcast the transaction ({}).", new_transaction.txid());
             return Ok(());
         }
-        crate::monitoring::increment_dkg_votes_submitted();
         // For all Pox-4 epochs onwards, broadcast the results also to stackerDB for other signers/miners to observe
         signer_transactions.push(new_transaction);
         let signer_message = SignerMessage::Transactions(signer_transactions);
         self.stackerdb.send_message_with_retry(signer_message)?;
+        crate::monitoring::increment_dkg_votes_submitted();
         info!("{self}: Broadcasted DKG vote transaction ({txid}) to stacker DB");
         Ok(())
     }