ChenSammi commented on code in PR #10875:
URL: https://github.com/apache/ozone/pull/10875#discussion_r3656909910
##########
hadoop-ozone/s3gateway/src/main/java/org/apache/hadoop/ozone/s3sts/S3STSEndpoint.java:
##########
@@ -128,35 +148,37 @@ public Response get(
@QueryParam("Version") String version,
@QueryParam("Policy") String awsIamSessionPolicy) throws OS3Exception {
- return handleSTSRequest(action, roleArn, roleSessionName, durationSeconds,
version, awsIamSessionPolicy);
+ return handleSTSRequest(
+ getQueryParameters().keySet(), action, roleArn, roleSessionName,
durationSeconds, version, awsIamSessionPolicy);
}
/**
* STS endpoint that handles POST requests with form data.
* AWS STS typically uses POST requests with form-encoded parameters.
*
- * @param action The STS action to perform
- * @param roleArn The ARN of the role to assume
- * @param roleSessionName Session name for the role
- * @param durationSeconds Duration of the token validity
- * @param version AWS STS API version
+ * @param form form-encoded request parameters
* @return Response containing STS response XML or error
*/
@POST
+ @Consumes(MediaType.APPLICATION_FORM_URLENCODED)
Review Comment:
It looks like AWS STS support ”application/x-www-form-urlencoded“ and
“application/x-amz-json-1.1”, we only support the first one?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
