fix: do not add extra 's' if the protocol is not 'http'

Signed-off-by: Ilya Buziuk <[email protected]>
eclipse-che · Oct 10, 2023 · 133ff5e · 133ff5e
1 parent 1bbcce6
commit 133ff5e
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/tools/build/src/devfile/meta-yaml-to-devfile-yaml.ts b/tools/build/src/devfile/meta-yaml-to-devfile-yaml.ts
@@ -131,12 +131,12 @@ export class MetaYamlToDevfileYaml {
           devfileEndpoint.attributes['type'] = 'main';
         }
 
-        // if it's secured, remove secure option for now but add extra s on the protocol
+        // if it's secured, remove secure option for now but change protocol from 'http' to 'https'
         if (devfileEndpoint.attributes && devfileEndpoint.attributes.secure === true) {
           devfileEndpoint.secure = false;
           delete devfileEndpoint.attributes['secure'];
-          // add extra s
-          if (devfileEndpoint.attributes.protocol) {
+          // switch 'http' to 'https'
+          if (devfileEndpoint.attributes.protocol && devfileEndpoint.attributes.protocol === "http") {
             devfileEndpoint.attributes.protocol = `${devfileEndpoint.attributes.protocol}s`;
           }
         }