-
Notifications
You must be signed in to change notification settings - Fork 50
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
Update configuration structure. #205
Comments
I suggest the default port to be 8888 rather than 80 since 80 isn't available on windows. |
Perhaps secureREST, httpsecure, sessioncookieid, sessioncookiesecret could be grouped. And authentication and guest might be related too. |
I'd suggest consistency in capitalization. |
Also, are all configuration keys listed? Or are we missing some? |
"guest" really means "allow anonymous access". It could stand to be renamed |
I suggest to have sub keys per features:
|
I think "autorecconnect" should be "autoreconnect" On Wed, Mar 4, 2015 at 10:26 AM Zsolt Lattmann notifications@github.com
|
Guys based on your comments I've updated the suggested config. (Some of the options aren't available yet.) |
Refactored and deal with node_worker timeouts better.
Conflicts: src/common/storage/serverstorage.js test/common/core/corerel.spec.js
var config = {
}
The text was updated successfully, but these errors were encountered: