[ 
https://issues.apache.org/jira/browse/HADOOP-18975?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17803393#comment-17803393
 ] 

ASF GitHub Bot commented on HADOOP-18975:
-----------------------------------------

ahmarsuhail commented on code in PR #6277:
URL: https://github.com/apache/hadoop/pull/6277#discussion_r1442459388


##########
hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/DefaultS3ClientFactory.java:
##########
@@ -165,6 +175,8 @@ private <BuilderT extends S3BaseClientBuilder<BuilderT, 
ClientT>, ClientT> Build
             .pathStyleAccessEnabled(parameters.isPathStyleAccess())
             .build();
 
+    builder.fipsEnabled(parameters.isFipsEnabled());

Review Comment:
   we already set this on line 265, so can remove from here.





> AWS SDK v2:  extend support for FIPS endpoints
> ----------------------------------------------
>
>                 Key: HADOOP-18975
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18975
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>    Affects Versions: 3.4.0
>            Reporter: Steve Loughran
>            Assignee: Steve Loughran
>            Priority: Major
>              Labels: pull-request-available
>
> v1 SDK supported FIPS just by changing the endpoint.
> Now we have a new builder setting to use.
> * add new  fs.s3a.endpoint.fips option
> * pass it down
> * test



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to