minio / minio-cpp

MinIO C++ Client SDK for Amazon S3 Compatible Cloud Storage
https://minio-cpp.min.io/
Apache License 2.0
139 stars 56 forks source link

Suggestion: Use std::ios::binary flag when opening files for upload #175

Closed HMS-Yukikaze closed 2 months ago

HMS-Yukikaze commented 2 months ago

Description The file is currently opened using file.open(args.filename);, which does not include the std::ios::binary flag. This causes problems when uploading non-text files (e.g., binary files like images or executables). To properly handle these types of files, it is recommended to open the file using file.open(args.filename, std::ios::binary);

Steps to Reproduce: up an *.psd file to the oss sever error in line:https://github.com/minio/minio-cpp/blob/40824d0872861b06965a032a714a8f4f9097fb32/src/utils.cc#L601 OS: Windows 11 22631.4037 Compiler:MSVC 2019 x64