Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

uri escape to comply with s3 #1280

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Conversation

prakashsvmx
Copy link
Member

uriEscape to comply with s3.
S3 percent-encodes some extra non-standard characters in a URI . So comply with S3.

//get stream

 try {
        //const result = await s3Client.putObject('test-bucket', `/sa-test-ba/KYC/楽天モバイル法人契約端末購入証明書_[21213213]_[20231113]_[BCDE ddnsf3 litelite 手帳型ケース シンプル ネイビー2])].pdf`,fileStream, stat.size )
        const result = await s3Client.putObject('test-bucket', `rfc3986/rfc3986-filename_test (123)äöü 🥳.txt`,fileStream, stat.size )
      } catch (err) {
        console.log("Error putObject ",  err.message, )
      }


const getPresignedUrl = async function(){

  const getUrl = await s3Client.presignedGetObject("test-bucket", `rfc3986/rfc3986-filename_test (123)äöü 🥳.txt`)
  console.log(getUrl)
}
getPresignedUrl()

@prakashsvmx prakashsvmx self-assigned this Apr 24, 2024
@prakashsvmx prakashsvmx merged commit 38a42b3 into minio:master Apr 25, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants