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

Added example DNS & SSL configuration #111

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jshair
Copy link

@jshair jshair commented Dec 28, 2023

Hi Team! First time doing a pull request! Working with Joe Mirabal who has a customer looking to use the CFT and SSL/TLS and Route 53. Ed Randall may create some additional documentation providing customers some guidance. In the meantime, David Rosenblum asked me to take a look.

So, here's an example of creating an SSL cert, using it on the listeners, and creating a DNS record within Route 53. I tested using our neo4j-field hosted zone within Route 53. I tested deploying only a single Neo4j instance. Not a cluster. But I did not change anything within the script for deploying Neo4j. The certs are applied to the NLB while the Neo4j ports remain as the default ports.

Hit me up on slack if any questions.

@harshitsinghvi22
Copy link
Contributor

@jshair thanks for raising the PR.
can you put the entire code under custom templates directory ?
https://github.com/neo4j-partners/amazon-cloud-formation-neo4j/tree/main/custom-templates
We would not want to merge this in the main template which is used in the marketplace listing. Hence , we have a directory called "Custom Templates" where we push all the R&D type templates

@jshair
Copy link
Author

jshair commented Jan 10, 2024

Hi @harshitsinghvi22 and @edrandall-dev! I finally got around to moving files and polishing up the associated README. Take a gander when you get a chance.

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