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

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

Closed
wants to merge 1 commit into from

Conversation

utterstep
Copy link

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

@utterstep
Copy link
Author

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

@oddgrd
Copy link
Contributor

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
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
Contributor

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