graph-app-kit
graph-app-kit copied to clipboard
[ENH] Optional creds via CloudFormation param
Instead of doing the public IP lookup (https://github.com/graphistry/graph-app-kit/blob/master/src/bootstraps/neptune/graphistry.sh#L45), which does not work in some modes, we should support:
- an optional CloudFormation setting of initial admin's user/password
- ... be sure to skip the IP lookup as it may fail in locked-down accounts
- when not provided, fallback to lookup
See: https://github.com/graphistry/graph-app-kit/issues/32, https://github.com/graphistry/graph-app-kit/issues/31, https://github.com/graphistry/graph-app-kit/issues/28
@lmeyerov Amazing thanks!
How long can we except for this enhancement to be ready?
I would like to start visualizing my production data but can't due to security concerns
@aminsshahab I'm checking w/ our AWS collaborators for guidance -- assuming not complicated, maybe end-of-week / next?
I think the private subnet thing is:
- If creds specified, remove INSTANCE ID public IP lookup and its use for default creds:
-
https://github.com/graphistry/graph-app-kit/blob/1e93ce164c26787a4a8673c9f73cb9573d906883/src/bootstraps/core/graphistry.sh#L23
-
https://github.com/graphistry/graph-app-kit/blob/e882fb094b4712c138cfdf9329bc077e90d658b6/src/bootstraps/neptune/graphistry.sh#L35 -> https://github.com/graphistry/graph-app-kit/blob/1e93ce164c26787a4a8673c9f73cb9573d906883/src/bootstraps/scripts/graphistry-service-account.sh#L20
- Add optional provisioning of user-specific creds:
-
Load param. I didn't see a clear way to do secrets (AWS Secrets Manager) that wasn't an ordeal for everyone involved, so probably just
noecho: https://github.com/graphistry/graph-app-kit/blob/1e93ce164c26787a4a8673c9f73cb9573d906883/src/bootstraps/core/graphistry.yml#L18 -
https://github.com/graphistry/graph-app-kit/blob/1e93ce164c26787a4a8673c9f73cb9573d906883/src/bootstraps/scripts/graphistry-service-account.sh#L20
-
Ensure the Graphistry boot scripts (graphistry-compose.service, something like that) isn't also doing the INSTANCE thing, and if it is, simply resume after it (expectedly) fails
However, I'm fuzzy on the surrounding public/private VPC stuff. Ex: If we did the above change, and you picked a private VPC... would that be enough.. or is there more needed?
FWIW, are you connecting this w/ Neptune or anything?
@lmeyerov My apologies for the late response.
A VPC is an isolated virtual network within AWS. For an EC2 instance you can decide to choose a public subnet which exposes the instance to the internet, or you can choose to only have a private subnet which only allows the instance to be accessible through your VPC.
Within the cloud-formation stack, you can have the user choose a private subnet or a public subnet. If the user chooses a private subnet then all you would need to do is use the private ip instead of the public ip.
Also, yes I will be using this to connect to Neptune.
Great -- and how would web browsers access your graph-app-kit instance... say via an AWS-friendly VPN that allows access to the private IP, or maybe a public ELB that has access to the private IP?
I'm trying to suss out if there's anything else we should be adding, or it's sufficient to remove to the disallowed instance-id.sh call. Ex: Should we also provide an optional param for doing an ELB hookup of some sort.
@lmeyerov Web browsers can access the instance using a VPN that has access to the VPC, which is the most straight forward way to do it.