Skip to content

Conversation

danielfrankcom
Copy link
Contributor

Summary

Adds a new dsql plugin which creates authentication tokens for Amazon Aurora DSQL.

Description

Changes include the added dsql plugin, which is an extension of the existing IamAuthConnectionPlugin with a custom token generator. Both new files are unit tested.

A new dependency was added which customers will also need if using the plugin. Requirements will be documented in an upcoming change.

Credit to @karenc-bq for the initial plugin implementation.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@danielfrankcom danielfrankcom changed the title Add plugin for DSQL auth tokens add plugin for DSQL auth tokens Jul 16, 2025
@danielfrankcom danielfrankcom force-pushed the dev/frankcom/dsql-plugin branch from 2bd92c9 to a53fa62 Compare July 16, 2025 18:17
@danielfrankcom
Copy link
Contributor Author

Latest change is a simple rebase to update the branch with the latest main.

@danielfrankcom danielfrankcom force-pushed the dev/frankcom/dsql-plugin branch from a53fa62 to e07c358 Compare July 16, 2025 19:54
@danielfrankcom
Copy link
Contributor Author

Update to e07c358 addresses a few issues:

  • Avoid wildcard imports as requested
  • Add support for extracting region from connection string for DSQL
  • Fix test issue caused by rebase

@danielfrankcom
Copy link
Contributor Author

Update to a7378b3 changes the dsql plugin name to iamDsql.

@danielfrankcom
Copy link
Contributor Author

Update to bb1f7c5 uses a static mock instead of a test-only constructor.

@danielfrankcom danielfrankcom force-pushed the dev/frankcom/dsql-plugin branch from bb1f7c5 to a1c6593 Compare July 21, 2025 17:42
@danielfrankcom
Copy link
Contributor Author

Update to a1c6593 fixes some formatting issues which were flagged by the GitHub actions.

@danielfrankcom danielfrankcom force-pushed the dev/frankcom/dsql-plugin branch from a1c6593 to 7c9c3fc Compare July 21, 2025 18:04
@danielfrankcom
Copy link
Contributor Author

Update to 7c9c3fc removes a unit test dependency on the system AWS credentials provider.

@danielfrankcom danielfrankcom force-pushed the dev/frankcom/dsql-plugin branch from 7c9c3fc to 3e6bd2a Compare July 23, 2025 17:22
@danielfrankcom
Copy link
Contributor Author

Update to 3e6bd2a addresses a comment by adding a user-facing warning message if the DSQL SDK dependency is missing.

@danielfrankcom danielfrankcom force-pushed the dev/frankcom/dsql-plugin branch from 3e6bd2a to 4dfd2a0 Compare July 23, 2025 17:59
@danielfrankcom
Copy link
Contributor Author

Update to 4dfd2a0 addresses an offline comment from @deepakscomk about a possible error for subsequent calls to generateAuthenticationToken with different credentialsProvider or region.

@danielfrankcom danielfrankcom force-pushed the dev/frankcom/dsql-plugin branch from 4dfd2a0 to 8c3d0a7 Compare July 23, 2025 18:54
@danielfrankcom
Copy link
Contributor Author

Update to 8c3d0a7 performs a rebase to pull in a fix from main for a javadoc error.

@danielfrankcom danielfrankcom force-pushed the dev/frankcom/dsql-plugin branch from 8c3d0a7 to b879640 Compare July 23, 2025 23:35
@karenc-bq karenc-bq changed the title add plugin for DSQL auth tokens feat: add plugin for DSQL auth tokens Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants