[GitHub] [incubator-druid] himanshug commented on a change in pull request #7674: add s3 authentication method informations

[GitBox <gi...@apache.org>]

himanshug commented on a change in pull request #7674: add s3 authentication method informations
URL: https://github.com/apache/incubator-druid/pull/7674#discussion_r284835991
 
 

 ##########
 File path: docs/content/development/extensions-core/s3.md
 ##########
 @@ -65,6 +65,23 @@ As an example, to set the region to 'us-east-1' through system properties:
 `s3:GetObject` and `s3:PutObject` are basically required for pushing/loading segments to/from S3.
 If `druid.storage.disableAcl` is set to `false`, then `s3:GetBucketAcl` and `s3:PutObjectAcl` are additionally required to set ACL for objects.
 
+### S3 authentication methods
+
+To connect to your S3 bucket (whether deep storage bucket or source bucket), Druid use the following credentials providers chain
+
+|order|type|details|
+|--------|-----------|-------|
+|1|Druid config file|Based on your runtime.properties if it contains values `druid.s3.accessKey` and `druid.s3.secretKey` |
+|2|Custom properties file| Based on custom properties file where you can supply `sessionToken`, `accessKey` and `secretKey` values|
 
 Review comment:
   would be nice if you document 'druid.s3.fileSessionCredentials' as well or else this point is harder to understand.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org