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

Use nconf for configuration management #24

Open
Marak opened this issue Jul 7, 2012 · 2 comments
Open

Use nconf for configuration management #24

Marak opened this issue Jul 7, 2012 · 2 comments
Labels
major version Major, potentially breaking, change

Comments

@Marak
Copy link
Contributor

Marak commented Jul 7, 2012

We should replace the current argument parsing with optimist and replace it with nconf.

This will allow http-server to be configured via the command line, or a config.json file, or any other transport nconf supports.

@BigBlueHat
Copy link
Member

We'll need to re-do the work in #131, but getting onto something more current/used would help us avoid likely security issues in the future.

@thornjad
Copy link
Member

I still support moving to nconf, though there have been significant changes since #131 which probably require a complete restart of that work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major version Major, potentially breaking, change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants