[GitHub] [hadoop] mukund-thakur commented on a change in pull request #2777: HADOOP-17476. ITestAssumeRole.testAssumeRoleBadInnerAuth failure.

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

mukund-thakur commented on a change in pull request #2777:
URL: https://github.com/apache/hadoop/pull/2777#discussion_r595760826



##########
File path: hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/auth/ITestAssumeRole.java
##########
@@ -255,8 +255,7 @@ public void testAssumeRoleBadInnerAuth() throws Exception {
     conf.set(SECRET_KEY, "not secret");
     expectFileSystemCreateFailure(conf,
         AWSBadRequestException.class,
-        "not a valid " +
-        "key=value pair (missing equal-sign) in Authorization header");
+        "IncompleteSignature");

Review comment:
       As per the commit message "Removes string match so change in AWS S3 error message doesn't cause the test to fail", the complete string matching is to be removed right?
   
   




----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org