satrs-shared/Cargo.toml

@@ -25,7 +25,7 @@ optional = true
 version = ">0.9"
 default-features = false
 git = "https://egit.irs.uni-stuttgart.de/rust/spacepackets"
-branch = "all-cfdp-updates"
+branch = "main"
 
 [features]
 serde = ["dep:serde", "spacepackets/serde"]

satrs/Cargo.toml

@@ -31,7 +31,7 @@ default-features = false
 version = "0.12"
 default-features = false
 git = "https://egit.irs.uni-stuttgart.de/rust/spacepackets"
-branch = "eof-pdu-update"
+branch = "main"
 
 [dependencies.cobs]
 git = "https://github.com/robamu/cobs.rs.git"

satrs/src/cfdp/source.rs

@@ -352,6 +352,7 @@ impl<
         }
         if self.state_helper.step == TransactionStep::NoticeOfCompletion {
             self.notice_of_completion(cfdp_user);
+            self.reset();
             /*
             def _notice_of_completion(self):
                 if self.cfg.indication_cfg.transaction_finished_indication_required:
@@ -709,6 +710,8 @@ impl<
 
     /// This function is public to allow completely resetting the handler, but it is explicitely
     /// discouraged to do this. CFDP has mechanism to detect issues and errors on itself.
+    /// Resetting the handler might interfere with these mechanisms and lead to unexpected
+    /// behaviour.
     pub fn reset(&mut self) {
         self.state_helper = Default::default();
         self.tstate = None;
@@ -767,7 +770,7 @@ mod tests {
         let fault_handler = TestFaultHandler::default();
         let test_sender = TestCfdpSender::default();
         let source_handler = default_source_handler(fault_handler, test_sender);
-        // assert!(dest_handler.transmission_mode().is_none());
+        assert!(source_handler.transmission_mode().is_none());
         // assert!(fault_handler.all_queues_empty());
     }
 }