Skip to content

Commit e35ad91

Browse files
authored
feat, close Da connecton is the server receive a bad node state (#1248)
1 parent c454459 commit e35ad91

File tree

1 file changed

+4
-2
lines changed
  • protocol-units/da-sequencer/node/src

1 file changed

+4
-2
lines changed

protocol-units/da-sequencer/node/src/server.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -273,7 +273,8 @@ impl DaSequencerNodeService for DaSequencerNode {
273273
) -> Result<tonic::Response<BatchWriteResponse>, tonic::Status> {
274274
if self.main_node_verifying_key.is_none() {
275275
tracing::warn!("Receive a node state and no verifying key is defined.");
276-
return Ok(tonic::Response::new(BatchWriteResponse { answer: false }));
276+
//return Ok(tonic::Response::new(BatchWriteResponse { answer: false }));
277+
return Err(tonic::Status::internal("Send state No verification"));
277278
}
278279
let state_data = request.into_inner();
279280

@@ -296,7 +297,8 @@ impl DaSequencerNodeService for DaSequencerNode {
296297
//unwrap tested just before
297298
if let Err(err) = self.main_node_verifying_key.as_ref().unwrap().verify(&data, &signature) {
298299
tracing::warn!("Grpc send_state called with a wrong signature : {err}");
299-
return Ok(tonic::Response::new(BatchWriteResponse { answer: false }));
300+
//return Ok(tonic::Response::new(BatchWriteResponse { answer: false }));
301+
return Err(tonic::Status::internal("Send state: Bad state"));
300302
}
301303

302304
let state =

0 commit comments

Comments
 (0)