-
Notifications
You must be signed in to change notification settings - Fork 5
BP-2222: Custom installation (initial tech writer review) #142
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
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
See minor suggestions/edits
| The following example uses port `7575` instead of the default `7474` port: | ||
|
|
||
| ```yaml | ||
| - 0.0.0.0:${NEO4J_WEB_PORT:-7575}:7474 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The IP should be 127.0.0.1
Using 0.0.0.0 bind the port to ALL interfaces and would expose bhce outside of localhost.
I think we should add a section that explains this. But I would do it with the web interface/port 8080
If you want, I can add the section, or you can do it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll defer to you on updating and adding a section to explain.
Updated bind_addr and NEO4J_WEB_PORT to use 127.0.0.1 instead of 0.0.0.0 for better security.
Purpose
This pull request (PR) proposes editorial and style revisions to the new custom installation page written by @Scoubi's in the
installbranch.Staging
https://specterops-bp-2222-custom-install-revised.mintlify.app/get-started/custom-installation