describeEndpoint Method Not Valid for Non-IOT [#3021268]

Problem/Motivation

The validate function that checks credentials when submitted on the form calls the describeEndpoint method to test the credentials. When not using IOT this doesn't make sense as the endpoint is provided.

Proposed resolution

Update the validate function to instead create a StsClient client instead of IotClient, and use the getCallerIdentity method to test the credentials instead of the endpoint.

Comment	File	Size	Author
#5	interdiff_2-5.txt	760 bytes	gaurav_drupal
#5	describe-endpoint-3021268-5.patch	1.55 KB	gaurav_drupal
#2	describe-endpoint-3021268-2.patch	1.47 KB	mikemadison

Comments

Comment #1

18 December 2018 at 16:56

mikemadison created an issue. See original summary.

Comment #2

mikemadison commented 18 December 2018 at 16:58

Status	File	Size
new	describe-endpoint-3021268-2.patch	1.47 KB

Comment #3

mikemadison commented 18 December 2018 at 16:59

Status:

Active

» Needs review

Comment #4

pixlkat commented 19 December 2018 at 21:01

This patch worked well for me -- the config form saves without error, and our IoT client is able to communicate normally.

My only comment about the patch is that if you are switching to the StsClient, you should be catching an Aws\Sts\Exception\StsException instead. The exception thrown if you enter invalid credentials on the form is not caught by the existing code.