Skip to content

Backport 1.5.x - Conditionally overwrite TLS parameters for MySQL secrets engine (#9729)#10073

Merged
Valarissa merged 1 commit into
release/1.5.xfrom
backport-pr-9729
Oct 1, 2020
Merged

Backport 1.5.x - Conditionally overwrite TLS parameters for MySQL secrets engine (#9729)#10073
Valarissa merged 1 commit into
release/1.5.xfrom
backport-pr-9729

Conversation

@Valarissa
Copy link
Copy Markdown

  • Conditionally overwrite TLS parameters in MySQL DSN

Overwrite MySQL TLS configuration in MySQL DSN only if have tls_ca or tls_certificate_key set
Current logic always overwrites it

  • Add test for MySQL DSN with a valid TLS parameter in query string

* Conditionally overwrite TLS parameters in MySQL DSN

Overwrite MySQL TLS configuration in MySQL DSN only if have `tls_ca` or `tls_certificate_key` set
Current logic always overwrites it

* Add test for MySQL DSN with a valid TLS parameter in query string
@Valarissa Valarissa added this to the 1.5.5 milestone Oct 1, 2020
@Valarissa Valarissa requested a review from a team October 1, 2020 18:53
Copy link
Copy Markdown
Contributor

@jasonodonnell jasonodonnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Valarissa Valarissa merged commit 4999336 into release/1.5.x Oct 1, 2020
tsaarni pushed a commit to Nordix/vault that referenced this pull request Oct 23, 2025
…ashicorp#9944) (hashicorp#10073) (hashicorp#10080)

* normalize token type for ns_token auth mounts

* add changelog

* also check type is supported in route and add test coverage

* Apply suggestion from @hellobontempo

* update test coverage to expect array

* update tests

Co-authored-by: claire bontempo <68122737+hellobontempo@users.noreply.github.com>
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.

3 participants