wandb / terraform-aws-wandb

A terraform module for deploying Weights & Biases on AWS.
Apache License 2.0
17 stars 19 forks source link

Remove "/usr/local/bin/local password" #267

Open vijay-wandb opened 2 months ago

vijay-wandb commented 2 months ago

Today: wandb-local (0.56) includes the command /usr/local/bin/local password. The command was used in the past when passwords were stored in a htpasswd file in /vol/env. The command is still working. Users can successfully login after is has been executed. Is the application sourcing htpasswd file and database for passwords or why is it still working?

Expectation: If it was a hack from the past and should not be used, we should remove it entirely.

abhinavg6 commented 2 months ago

@elainaRenee - This is a fair ask, right? Asking if this is something we could have consultants own at some point, or should it be addressed by us directly?

elainaRenee commented 3 weeks ago

Yes we had spoken about removing this. The code path isn't well maintained anymore and using this might cause unexpected issues.