fix(s3): make data integrity protections opt-in

kesselb · backportbot[bot] · commit fbe3252a8b6d · 2025-12-03T09:14:30.000Z
Signed-off-by: Daniel Kesselberg &lt;mail@danielkesselberg.de&gt;

[skip ci]
diff --git a/lib/private/Files/ObjectStore/S3ConnectionTrait.php b/lib/private/Files/ObjectStore/S3ConnectionTrait.php
@@ -127,10 +127,14 @@ public function getConnection() {
 
 		if (isset($this->params['request_checksum_calculation'])) {
 			$options['request_checksum_calculation'] = $this->params['request_checksum_calculation'];
+		} else {
+			$options['request_checksum_calculation'] = 'when_required';
 		}
 
 		if (isset($this->params['response_checksum_validation'])) {
 			$options['response_checksum_validation'] = $this->params['response_checksum_validation'];
+		} else {
+			$options['response_checksum_validation'] = 'when_required';
 		}
 
 		if ($this->getProxy()) {

Original file line number	Diff line number	Diff line change
`@@ -127,10 +127,14 @@ public function getConnection() {`
`127`	`127`
`128`	`128`	`if (isset($this->params['request_checksum_calculation'])) {`
`129`	`129`	`$options['request_checksum_calculation'] = $this->params['request_checksum_calculation'];`
	`130`	`+ } else {`
	`131`	`+ $options['request_checksum_calculation'] = 'when_required';`
`130`	`132`	`}`
`131`	`133`
`132`	`134`	`if (isset($this->params['response_checksum_validation'])) {`
`133`	`135`	`$options['response_checksum_validation'] = $this->params['response_checksum_validation'];`
	`136`	`+ } else {`
	`137`	`+ $options['response_checksum_validation'] = 'when_required';`
`134`	`138`	`}`
`135`	`139`
`136`	`140`	`if ($this->getProxy()) {`