ripplehang commented on PR #43601: URL: https://github.com/apache/arrow/pull/43601#issuecomment-2325830653
> In your PR, it appears that `S3Options.sse_customer_key` expects a base64-encoded private key. Instead, why not expect the raw 256-bit key? @pitrou though the boto3 exposed the 32-bytes key, the offcial c++ aws sdk expected the base64 encoded key, I would prefer to just passthrough the input to the aws sdk. Meanwhile, during the integration between the arrow and my own project, using the base64encoded key may be a little more easy for integration/debug/troubleshooting purpose. Anyway, if you insist on exposing the 32 bit raw key, I'm also fine with that. @kou @mapleFU any insights? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
