Skip to content

Commit

Permalink
Merge pull request #1748 from tinect/fix/updateSimpleS3
Browse files Browse the repository at this point in the history
fix: [AsyncAwsS3] update simpleS3 for test and adjust getPresignedUrl declaration
  • Loading branch information
frankdejonge authored Feb 4, 2024
2 parents de9f800 + f9cb517 commit 7754a6a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/AsyncAwsS3/S3ClientStub.php
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ public function getUrl(string $bucket, string $key): string
return $this->actualClient->getUrl($bucket, $key);
}

public function getPresignedUrl(string $bucket, string $key, ?DateTimeImmutable $expires = null): string
public function getPresignedUrl(string $bucket, string $key, ?DateTimeImmutable $expires = null, ?string $versionId = null): string
{
return $this->actualClient->getPresignedUrl($bucket, $key, $expires);
}
Expand Down
5 changes: 2 additions & 3 deletions src/AsyncAwsS3/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,10 @@
"async-aws/s3": "^1.5 || ^2.0"
},
"require-dev": {
"async-aws/simple-s3": "^1.1 || ^2.0"
"async-aws/simple-s3": "^2.1"
},
"conflict": {
"symfony/http-client": "<5.2",
"async-aws/simple-s3": "<1.1"
"symfony/http-client": "<5.2"
},
"license": "MIT",
"authors": [
Expand Down

0 comments on commit 7754a6a

Please sign in to comment.