Skip to content

add example using rustls, without native bindings to openssl#43

Closed
utterstep wants to merge 1 commit intoshuttle-hq:mainfrom
utterstep:postgres-rustls-example
Closed

add example using rustls, without native bindings to openssl#43
utterstep wants to merge 1 commit intoshuttle-hq:mainfrom
utterstep:postgres-rustls-example

Conversation

@utterstep
Copy link
Copy Markdown

Simple copy of poem/postgres, using rustls instead of native-tls

@utterstep
Copy link
Copy Markdown
Author

Should not be merged before shuttle-hq/shuttle#870

@oddgrd
Copy link
Copy Markdown
Collaborator

oddgrd commented May 8, 2023

I'm a bit on the fence on whether or not we need a full example for this 🤔 What do you think? Either way it would be great if you could update the docs on shuttle-shared-db too (we have a bit of duplication of docs 😅): https://github.com/shuttle-hq/shuttle-docs/blob/main/resources/shuttle-shared-db.mdx

@utterstep
Copy link
Copy Markdown
Author

Yeah, I think it is not offering much on it's own.
Main point of this PR was to show the code I used to check the implementation, tbh :)

Sure, I can close this PR and update docs!

utterstep added a commit to utterstep/shuttle-docs that referenced this pull request May 8, 2023
Update docs, reflect the `rustls` implementation option in  `shuttle-shared-db`.

Re: shuttle-hq/shuttle-examples#43
Re: shuttle-hq/shuttle#870

cc @oddgrd
oddgrd pushed a commit to shuttle-hq/shuttle-docs that referenced this pull request May 9, 2023
Update docs, reflect the `rustls` implementation option in  `shuttle-shared-db`.

Re: shuttle-hq/shuttle-examples#43
Re: shuttle-hq/shuttle#870

cc @oddgrd
@oddgrd
Copy link
Copy Markdown
Collaborator

oddgrd commented May 9, 2023

Main point of this PR was to show the code I used to check the implementation, tbh :)

Thanks for that!

@oddgrd oddgrd closed this May 9, 2023
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.

2 participants