Error logging improvements for #124

Mermade · May 10, 2018 · 059c64b · 059c64b
1 parent f6ad52e
commit 059c64b
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 6 deletions.
diff --git a/httpsnippetGenerator.js b/httpsnippetGenerator.js
@@ -11,7 +11,7 @@ function generate(target, client, data) {
     }
     catch (ex) {
         console.warn('Error generating code sample using httpsnippet for', target, client);
-        if (data.options.verbose) console.warn(ex.message);
+        if (data.options.verbose) console.warn(ex);
         return '';
     }
 }

diff --git a/openapi3.js b/openapi3.js
@@ -184,7 +184,7 @@ function getParameters(data) {
         if (param.description === 'undefined') { // yes, the string
             param.description = '';
         }
-        if (typeof param.description !== 'undefined') {
+        if ((typeof param.description !== 'undefined') && (typeof param.description === 'string')) {
             param.shortDesc = param.description.split('\n')[0];
             if (param.shortDesc !== param.description) data.longDescs = true;
         }

diff --git a/widdershins.js b/widdershins.js
@@ -90,12 +90,17 @@ function doit(s) {
     }
 
     converter.convert(api,options,function(err,output){
-        var outfile = argv.outfile||argv._[1];
-        if (outfile) {
-            fs.writeFileSync(path.resolve(outfile),output,'utf8');
+        if (err) {
+            console.warn(err);
         }
         else {
-            console.log(output);
+            var outfile = argv.outfile||argv._[1];
+            if (outfile) {
+                fs.writeFileSync(path.resolve(outfile),output,'utf8');
+            }
+            else {
+                console.log(output);
+            }
         }
     });
 }