Fix swagger host setting in order to add support for the live environ…

…ment. (#35)
release-notes · Jan 6, 2018 · e3ed890 · e3ed890
1 parent 8a176d4
commit e3ed890
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/api/openapi-v1.js b/api/openapi-v1.js
@@ -1,11 +1,11 @@
-module.exports = function({ version }) {
+module.exports = function({ version, host }) {
   const HANDLE_REGEX = '[a-zA-Z0-9][a-zA-Z0-9\\-]*[a-zA-Z0-9]';
   const SCOPE_REGEX = `@?${HANDLE_REGEX}`;
 
   return {
     swagger: '2.0',
     basePath: '/api/v1',
-    host: "localhost:8080",
+    host,
     info: {
       title: 'The Release Notes API',
       version: version,

diff --git a/config/application.js b/config/application.js
@@ -5,7 +5,8 @@ const svgEmbed = require('../lib/svg-embed');
 
 const APP_PATH = `${__dirname}/..`;
 const env = process.env;
-const APP_BASE_URL = env.BASE_URL || 'https://release-notes.com';
+const APP_HOST = env.HOST || 'release-notes.com';
+const APP_BASE_URL = env.BASE_URL || `https://${APP_HOST}`;
 
 module.exports = {
   app: {
@@ -109,6 +110,7 @@ module.exports = {
 
   api: {
     version: '0.0.0',
+    host: APP_HOST,
   },
 
   apiController: {

diff --git a/services/ApiService.js b/services/ApiService.js
@@ -14,12 +14,13 @@ class ApiService extends Service {
 
   bind(expressApp) {
     const version = this.serviceConfig.get('version');
+    const host = this.serviceConfig.get('host');
     const serviceManager = this.serviceManager;
     const authTokenRepository = serviceManager.get('authTokenRepository');
 
     expressOpenapi.initialize({
       app: expressApp,
-      apiDoc: apiDocV1({ version }),
+      apiDoc: apiDocV1({ version, host }),
       paths: './api/v1',
       docsPath: '/api-spec',
       exposeApiDocs: true,