Fixed : multiple url-encoding, incorrect querystring appender

Author: mrin9

dist/rapidoc-min.js

@@ -1,6 +1,6 @@
 {
   "name": "rapidoc",
-  "version": "5.0.4",
+  "version": "5.0.5",
   "description": "RapiDoc - Open API spec viewer with built in console",
   "author": "Mrinmoy Majumdar <[email protected]>",
   "repository": {

dist/rapidoc-min.js.gz

@@ -224,7 +224,7 @@ export default class ApiRequest extends LitElement {
         
           </td>
           <td colspan="2" style="border:none; margin-top:0; padding:0 5px;"> 
-            <span class="m-markdown-small">${unsafeHTML(marked(param.description))}</span>
+            <span class="m-markdown-small">${unsafeHTML(marked(param.description || ""))}</span>
           </td>
         </tr>`
         :``}
@@ -251,7 +251,7 @@ export default class ApiRequest extends LitElement {
 
     let mimeReqCount=0;
     let shortMimeTypes={};
-    let bodyDescrHtml = this.request_body.description? html`<div class="m-markdown"> ${unsafeHTML(marked(this.request_body.description))}</div>`:'';
+    let bodyDescrHtml = this.request_body.description? html`<div class="m-markdown"> ${unsafeHTML(marked(this.request_body.description || ""))}</div>`:'';
     let textareaExampleHtml='';
     let formDataHtml='';
     const formDataTableRows = [];
@@ -335,7 +335,7 @@ export default class ApiRequest extends LitElement {
             <tr>
               <td style="border:none"></td>
               <td colspan="2" style="border:none; margin-top:0; padding:0 5px;"> 
-                <span class="m-markdown-small">${unsafeHTML(marked(fieldSchema.description))}</span>
+                <span class="m-markdown-small">${unsafeHTML(marked(fieldSchema.description || ""))}</span>
               </td>
             </tr>`
             :``}
@@ -487,7 +487,7 @@ export default class ApiRequest extends LitElement {
     }
     //Generate URL using Path Params
     pathParamEls.map(function(el){
-	    fetchUrl = fetchUrl.replace("{"+el.dataset.pname+"}", encodeURIComponent(el.value));
+	    fetchUrl = fetchUrl.replace("{"+el.dataset.pname+"}", el.value);
     });
 
     //Submit Query Params
@@ -496,13 +496,13 @@ export default class ApiRequest extends LitElement {
       queryParamEls.map(function(el){
         if (el.dataset.array==='false'){
           if (el.value !== ''){
-            queryParam.append(el.dataset.pname, encodeURIComponent(el.value));
+            queryParam.append(el.dataset.pname, el.value);
           }
         }
         else {
           let vals = el.getValues();
           for(let v of vals){
-            queryParam.append(el.dataset.pname, encodeURIComponent(v));
+            queryParam.append(el.dataset.pname, v);
           }
         }
       })
@@ -511,7 +511,7 @@ export default class ApiRequest extends LitElement {
 
     // Add authentication Query-Param if provided 
     if (this.apiKeyValue && this.apiKeyName && this.apiKeyLocation==='query'){
-      fetchUrl = `${fetchUrl}&${this.apiKeyName}=${encodeURIComponent(this.apiKeyValue)}`;
+      fetchUrl = `${fetchUrl}${fetchUrl.includes("?")?'&':'?'}${this.apiKeyName}=${encodeURIComponent(this.apiKeyValue)}`;
     }
 
     //Final URL for API call

dist/rapidoc-min.js.map

@@ -179,7 +179,7 @@ export default class ApiResponse extends LitElement {
               <tr>
                 <td style="padding:0 12px;vertical-align: top;" class="regular-font-size"> ${v.name}</td> 
                 <td style="padding:0 12px;vertical-align: top; line-height:14px" class="descr-text small-font-size">
-                  <span class="m-markdown-small">${unsafeHTML(marked(v.description))}</span>
+                  <span class="m-markdown-small">${unsafeHTML(marked(v.description || "" ))}</span>
                   ${ (v.schema && v.schema.example)? html`<br/><span style="font-weight:bold">EXAMPLE:</span> ${v.schema.example}`:`` }
                 </td>
               </tr>

dist/rapidoc-min.js.map.gz

@@ -33,18 +33,18 @@ export default class EndPoint extends LitElement {
         <div class="path ${this.path.deprecated?'deprecated':''}"> ${this.path.path} </div>
         ${this.path.deprecated?html`<span style="font-size:12px; text-transform:uppercase; font-weight:bold; color:orangered; margin:2px 0 0 5px;"> deprecated </span>`:''}
         <div class="only-large-screen" style="min-width:60px; flex:1"></div>
-        <div class="m-markdown-small descr"> ${unsafeHTML(marked(this.path.summary))} </div>
+        <div class="m-markdown-small descr"> ${unsafeHTML(marked(this.path.summary || ""))} </div>
       </div>
       
       <!-- Endpoint Body -->
       ${this.path.expanded?html`
       <div class='body ${this.path.method}'>
         ${this.path.summary || this.path.description?html`
           <div class="summary">
-            <div class="m-markdown title">${unsafeHTML(marked(this.path.summary))}</div>
+            <div class="m-markdown title">${unsafeHTML(marked(this.path.summary || ""))}</div>
             ${this.path.summary !== this.path.description?html`
               <div class="m-markdown"> 
-                ${unsafeHTML(marked(this.path.description?this.path.description:''))}
+                ${unsafeHTML(marked(this.path.description || ""))}
               </div>`
             :''}  
           </div>`