unify websocket configuration points

This commit is contained in:
ansuz 2019-12-23 18:27:11 -05:00
parent a8b363dba0
commit 05699b2c94
3 changed files with 17 additions and 3 deletions

View File

@ -1,3 +1,17 @@
# J release (3.9.0)
## Goals
* simplify server configuration by dropping support for unused features
* drop `websocketPath` `websocketURL` and `useExternalWebsockets`
* use `externalWebsocketURL` to replace all of these
## Update notes
## Features
## Bug fixes
# IsolobodonPortoricensis release (3.8.0)
We had some trouble finding an extinct animal whose name started with "I", and we had to resort to using a scientific name.

View File

@ -107,7 +107,7 @@ module.exports = {
* Otherwise, leave this commented and your clients will use the default
* websocket (wss://yourdomain.com/cryptpad_websocket)
*/
//externalWebsocketPath: 'wss://api.yourdomain.com/cryptpad_websocket
//externalWebsocketURL: 'wss://api.yourdomain.com/cryptpad_websocket
/* CryptPad can be configured to send customized HTTP Headers
* These settings may vary widely depending on your needs

View File

@ -166,7 +166,7 @@ app.get('/api/config', function(req, res){
},
removeDonateButton: (config.removeDonateButton === true),
allowSubscriptions: (config.allowSubscriptions === true),
websocketPath: config.externalWebsocketPath,
websocketPath: config.externalWebsocketURL,
httpUnsafeOrigin: config.httpUnsafeOrigin.replace(/^\s*/, ''),
adminEmail: config.adminEmail,
adminKeys: admins,
@ -234,7 +234,7 @@ var nt = nThen(function (w) {
log = config.log = _log;
}));
}).nThen(function (w) {
if (config.externalWebsocketPath) {
if (config.externalWebsocketURL) {
// if you plan to use an external websocket server
// then you don't need to load any API services other than the logger.
// Just abort.