Merge pull request #264 from Unleash/disable-legacy-routes

Added option to disable legacy routes. Closes #244

docs/getting-started.md

@@ -35,8 +35,9 @@ unleash.start({
 
 Available unleash options includes:
 
-- databaseUrl 
-- port
+- **databaseUrl** - the postgress database url to connect to. Should include username/password. 
+- **port** - Which port should the unleash-server bind to?
+- **enableLegacyRoutes** (boolean) - allows you to turn on/off support for legacy routes to support older clients. 
 
 ## How do I configure the log output?
  

lib/options.js

@@ -9,6 +9,7 @@ const DEFAULT_OPTIONS = {
     port: process.env.HTTP_PORT || process.env.PORT || 4242,
     baseUriPath: process.env.BASE_URI_PATH || '',
     serverMetrics: true,
+    enableLegacyRoutes: true,
     publicFolder,
     enableRequestLogger: isDev(),
 };

lib/routes/index.js

@@ -38,7 +38,9 @@ exports.router = function(config) {
     // $root/features
     // $root/client/register
     // $root/client/metrics
-    router.use('/api/features', clientFeatures.router(config));
+    if (config.enableLegacyRoutes) {
+        router.use('/api/features', clientFeatures.router(config));
+    }
 
     return router;
 };

lib/routes/index.test.js

@@ -15,6 +15,7 @@ function getSetup() {
         baseUriPath: base,
         stores,
         eventBus,
+        enableLegacyRoutes: true,
     });
 
     return {