Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Allow for inlining client cert #342

Merged
merged 1 commit into from
Sep 9, 2023
Merged

Conversation

vr
Copy link
Contributor

@vr vr commented Aug 31, 2023

It is useful when you are using Google Cloud SQL and don't want to
be forced to use temporary files to load certificate

It is useful when you are using Google Cloud SQL and don't want to
be forced to use temporary files to load certificate
@vr
Copy link
Contributor Author

vr commented Aug 31, 2023

This should fix #253

@vr
Copy link
Contributor Author

vr commented Sep 5, 2023

May i ask @kylejohnson to look at this ? 🙏

@kylejohnson
Copy link
Contributor

If I am understanding this correctly, when setting sslinline = true, the values of clientcert.cert and clientcert.key are read directly from their set value? e.g.:

sslinline = true
clientcert {
  cert = "certvalue"
  key  = "keyvalue"
}

Copy link
Owner

@cyrilgdn cyrilgdn left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@cyrilgdn cyrilgdn merged commit 20086dc into cyrilgdn:master Sep 9, 2023
@cyrilgdn
Copy link
Owner

Released in v1.21.0

@vr vr deleted the sslinline branch September 11, 2023 07:34
dduportal referenced this pull request in jenkins-infra/azure Sep 23, 2023
<Actions>
<action
id="16c2f665ec8f38b263f8d1f01847791b753aaf4c15655cd80097752f2ecae989">
        <h3>Bump Terraform `postgresql` provider version</h3>
<details
id="1d9343c012f5434ac9fe8a98135bae3667b399259be16d9b14302ea3bd424a24">
            <summary>Update Terraform lock file</summary>
<p>&#34;cyrilgdn/postgresql&#34; updated from &#34;1.21.0&#34; to
&#34;1.21.0&#34; in file &#34;.terraform.lock.hcl&#34;</p>
            <details>
                <summary>1.21.0</summary>
<pre>Changelog retrieved
from:&#xA;&#x9;https://github.com/cyrilgdn/terraform-provider-postgresql/releases/tag/v1.21.0&#xA;##
What&#39;s Changed&#xD;&#xA;* Make function security, strictness,
volatility, and parallel safety configurable by @tommyzli in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/307&#xD;&#xA;*
postgresql_publication.markdown - table ordering by @RoseateSpoonbill in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/265&#xD;&#xA;*
Bump gocloud dependency to fix aws CA validation by @toadjaune in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/340&#xD;&#xA;*
feat: Allow for inlining client cert by @vr in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/342&#xD;&#xA;*
Update postgresql_extension.html.markdown by @tspearconquest in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/337&#xD;&#xA;*
Use uint32 for OID to stop conversion errors with pguint32 by @mat285 in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/324&#xD;&#xA;*
feat: Postgresql 15 support by @cyrilgdn in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/348&#xD;&#xA;*
Added MS Azure passwordless authentication by @librucha in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/346&#xD;&#xA;*
Add &#39;Injecting credentials&#39; section to provider docs by @rdarcy1
in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/318&#xD;&#xA;*
fixed escaping for postgresql user info by @librucha in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/344&#xD;&#xA;&#xD;&#xA;##
New Contributors&#xD;&#xA;* @tommyzli made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/307&#xD;&#xA;*
@RoseateSpoonbill made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/265&#xD;&#xA;*
@toadjaune made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/340&#xD;&#xA;*
@vr made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/342&#xD;&#xA;*
@tspearconquest made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/337&#xD;&#xA;*
@mat285 made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/324&#xD;&#xA;*
@librucha made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/346&#xD;&#xA;*
@rdarcy1 made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/318&#xD;&#xA;&#xD;&#xA;**Full
Changelog**:
https://github.com/cyrilgdn/terraform-provider-postgresql/compare/v1.20.0...v1.21.0</pre>
            </details>
        </details>
    </action>
</Actions>

---

<table>
  <tr>
    <td width="77">
<img src="https://www.updatecli.io/images/updatecli.png" alt="Updatecli
logo" width="50" height="50">
    </td>
    <td>
      <p>
Created automatically by <a
href="https://www.updatecli.io/">Updatecli</a>
      </p>
      <details><summary>Options:</summary>
        <br />
<p>Most of Updatecli configuration is done via <a
href="https://www.updatecli.io/docs/prologue/quick-start/">its
manifest(s)</a>.</p>
        <ul>
<li>If you close this pull request, Updatecli will automatically reopen
it, the next time it runs.</li>
<li>If you close this pull request and delete the base branch, Updatecli
will automatically recreate it, erasing all previous commits made.</li>
        </ul>
        <p>
Feel free to report any issues at <a
href="https://github.com/updatecli/updatecli/issues">github.com/updatecli/updatecli</a>.<br
/>
If you find this tool useful, do not hesitate to star <a
href="https://github.com/updatecli/updatecli/stargazers">our GitHub
repository</a> as a sign of appreciation, and/or to tell us directly on
our <a
href="https://matrix.to/#/#Updatecli_community:gitter.im">chat</a>!
        </p>
      </details>
    </td>
  </tr>
</table>

Co-authored-by: Jenkins Infra Bot (updatecli) <60776566+jenkins-infra-bot@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