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

[Bug] Local installation without SSL does not start #87

Closed
akuckartz opened this issue Apr 2, 2023 · 3 comments · Fixed by #108
Closed

[Bug] Local installation without SSL does not start #87

akuckartz opened this issue Apr 2, 2023 · 3 comments · Fixed by #108
Labels
bug Something isn't working

Comments

@akuckartz
Copy link

Graph Explorer can no longer be started without using SSL and certificates. The script https://github.com/aws/graph-explorer/blob/main/docker-entrypoint.sh then ends with exit 1. But according to https://github.com/aws/graph-explorer/blob/main/additionaldocs/development.md a docker container without GRAPH_EXP_HTTPS_CONNECTION should be accessible using http instead of https.

This results in significant additional efforts when AWS is to be installed and used only locally. And at the moment it is not clear if using SSL locally works. See #86

@akuckartz akuckartz added the bug Something isn't working label Apr 2, 2023
@akuckartz
Copy link
Author

This seems to be a duplicate of #41 which was already raised in January 2023.

@jackson-millard
Copy link
Contributor

Currently, you can't provide the environment variables in charge of SSL via the docker run command. Those need to be set in the .env folder inside the graph-explorer folder.

@michaelnchin
Copy link
Member

A pending change in #108 will add support for specifying the value of GRAPH_EXP_HTTPS_CONNECTION via env variable through docker run.

@michaelnchin michaelnchin linked a pull request Jul 19, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

3 participants