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

185765111 tls support #189

Merged
merged 1 commit into from
Dec 15, 2023
Merged

185765111 tls support #189

merged 1 commit into from
Dec 15, 2023

Conversation

dark5un
Copy link
Contributor

@dark5un dark5un commented Oct 2, 2023

The implementation of RDS broker does not use github.com/alphagov/paas-service-broker-base. For the time being I have "borrowed" and converted parts of it to make RDS-Broker support TLS connectivity.

How to test:

  • Check if the integration tests based on this branch have been successful
  • Use the artefact produced in a paas-cf deployment
  • Check that the broker works as expected on that deployment

@dark5un dark5un force-pushed the 185765111-tls-support branch from f1761ba to 49ce0e9 Compare October 10, 2023 07:04
@dark5un dark5un changed the title 185765111 tls support [WIP] 185765111 tls support Oct 16, 2023
@dark5un dark5un force-pushed the 185765111-tls-support branch from 49ce0e9 to a589c10 Compare October 18, 2023 15:18
@dark5un dark5un force-pushed the 185765111-tls-support branch from 95cf230 to aeb56f0 Compare October 27, 2023 07:51
@malcgds
Copy link
Contributor

malcgds commented Nov 27, 2023

I have seen that the integration tests worked. I have implemented the tls rds broker in dev05 and used the broker to create services, bind apps, unbind apps and delete services. It all looks good to me.

* Includes integration test covering selfsigned TLS
@dark5un dark5un force-pushed the 185765111-tls-support branch from e6f5f4d to 095d8ec Compare December 13, 2023 09:54
@dark5un dark5un merged commit 0c1015d into main Dec 15, 2023
3 checks passed
@dark5un dark5un deleted the 185765111-tls-support branch December 15, 2023 15:24
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