Merge pull request #66 from UMEssen/fix-storescp-pctx

Fix wrong transfer syntax recorded in file meta

Author: nickamzol

CHANGELOG.md

@@ -18,6 +18,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   single request (e.g. for every instance of a STOW-RS request).
 - Associations are no longer returned to the pool after a failed or partially completed message
   exchange, as their state is unknown.
+- Fix incorrect transfer syntax recorded in file meta when multiple presentation contexts were negotiated in STORE-SCP.
 
 ## Changed
 

src/backend/dimse/cstore/storescp.rs

@@ -92,10 +92,14 @@ impl StoreServiceClassProvider {
 
 		// Duration::MAX to indefinitely wait for incoming messages
 		while let Ok(message) = association.read_message(Duration::MAX).await {
+			let pctx_id = message
+				.presentation_context_id
+				.context("Missing presentation context id in received message")?;
 			let pctx = association
 				.presentation_contexts()
-				.first()
-				.context("No presentation context available")?;
+				.iter()
+				.find(|pctx| pctx.id == pctx_id)
+				.context("No negotiated presentation context matches the received message")?;
 			debug!(
 				"Used transfer syntax {} to read message",
 				pctx.transfer_syntax