Added Support for performing Assume role with Athena Connections

[thatguyfig]

Features and Changes

Provides the UI and backend for performing an assume role before running Athena queries. Since the getAthenaInstance function is used to obtain a client, this can also be used to interact with sts beforehand before returning the client with embedded credentials - this also incorporates the JavaScript v3 SDK.

I'm not to sure how the errors are propagated up the UI but it seems to work pretty well - so that should be one area to improve. Also the session is currently defaults to 900 seconds which could be too short for some beefy queries.

[thatguyfig]

There is another PR floating around that referenced my issue, but the code seems like it could work for the auto mode to assume another role for the app as a whole.

#2438

[thatguyfig]

#2442

[cleaton]

There is another PR floating around that referenced my issue, but the code seems completely half-baked

#2438

the upgraded SDK allows configuring credentials via aws cli config file, can add assume role configuration in there and auto will pick it up.

[thatguyfig]

There is another PR floating around that referenced my issue, but the code seems completely half-baked

#2438

the upgraded SDK allows configuring credentials via aws cli config file, can add assume role configuration in there and auto will pick it up.

Yes that is possible, but that doesn't offer a solution for when you have different connections in different accounts and need to be able to assume a specific role.

With this approach the assume role is tied to the connection and can be specified when you set it up so it can be created by the users themselves.

[cleaton]

There is another PR floating around that referenced my issue, but the code seems completely half-baked
#2438

the upgraded SDK allows configuring credentials via aws cli config file, can add assume role configuration in there and auto will pick it up.

Yes that is possible, but that doesn't offer a solution for when you have different connections in different accounts and need to be able to assume a specific role.

With this approach the assume role is tied to the connection and can be specified when you set it up so it can be created by the users themselves.

Sorry I think there is some misunderstanding here, I did not mean our PRs are equal or anything I only referenced it as a possible workaround as there was no PR mentioned there yet.

We want the same thing and this is a nice solution so I’m all for adding this instead, thank you for sharing it 👍

[jdorn]

Thanks for contributing! Left a few comments

[thatguyfig]

Thanks @jdorn - issues have been addressed, please could re-review?

[thatguyfig]

bump