misc: link documentation to fix sigv4a unsupported algorithm #1209
Conversation
0marperez
added
the
no-changelog
|
A new generated diff is ready to view.
|
|
A new generated diff is ready to view.
|
|
A new generated diff is ready to view.
|
|
A new generated diff is ready to view.
|
| @@ -23,7 +23,11 @@ public class UnsupportedSigningAlgorithmInterceptor : HttpInterceptor { | |||
| context.response.exceptionOrNull()?.let { | |||
| if (it is UnsupportedSigningAlgorithmException && it.signingAlgorithm == AwsSigningAlgorithm.SIGV4_ASYMMETRIC) { | |||
| return Result.failure( | |||
| it, // TODO: Add a message and link pointing to AWS SDK for Kotlin developer guide. | |||
| UnsupportedSigningAlgorithmException( | |||
| "${it.message} For more information on how to enable it with the CRT signer, please refer to: https://docs.aws.amazon.com/sdk-for-kotlin/latest/developer-guide/use-services-s3-mrap.html#mrap-s3client-config", | |||
There was a problem hiding this comment.
nit: ${it.message.ensureSuffix(".")}
There was a problem hiding this comment.
alternatively, ignore the it.message and just create your own message from scratch, since you know it will always be this:
SIGV4A support is not yet implemented for the default signer. For more information on how to enable it with the CRT signer, please refer to: https://docs.aws.amazon.com/sdk-for-kotlin/latest/developer-guide/use-services-s3-mrap.html#mrap-s3client-config
|
A new generated diff is ready to view.
|
|
A new generated diff is ready to view.
|
|
A new generated diff is ready to view.
|
|
|
A new generated diff is ready to view.
|
Issue #
closes #625
Description of changes
Adds a link to exception message on how to fix sigv4a unsupported signing algorithm
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.