Update play-dgraph-io.nginx.conf

dgraph-io / ratel

Dgraph Data Visualizer and Cluster Manager

Apache License 2.0

120 stars 49 forks source link

Closed MichelDiz closed 2 years ago

MichelDiz commented 2 years ago

Note: The Dg-Auth token is a user token. With no Admin rights.

skrdgraph commented 2 years ago

@MichelDiz some things I am not clear about (they are indirectly related to your PR)

https://github.com/dgraph-io/dgraph uses the secrets, I think we should explore that here for the tokens
Also, what's the deployment mechanism for this config change? Does it get auto-deployed? (if not, we should try to focus on that next)

I don't see any harm in merging this PR.

MichelDiz commented 2 years ago

We don't need to hide this specific token. But if you wanna add this, let's see.

There's no auto-deployment for now. The instance is manually configured. I see no sign of scripts in the EC2 instance.