Commit 7cbe60a4 authored by Girish Ramakrishnan's avatar Girish Ramakrishnan

Fix crash when only udp ports are defined

parent ded9a6e3
...@@ -1697,4 +1697,5 @@ ...@@ -1697,4 +1697,5 @@
* Fix issue where the icon for normal users was displayed incorrectly * Fix issue where the icon for normal users was displayed incorrectly
* Kill stuck backup processes after 12 hours and notify admins * Kill stuck backup processes after 12 hours and notify admins
* Reconfigure email apps when mail domain is added/removed * Reconfigure email apps when mail domain is added/removed
* Fix crash when only udp ports are defined
...@@ -180,7 +180,7 @@ function createSubcontainer(app, name, cmd, options, callback) { ...@@ -180,7 +180,7 @@ function createSubcontainer(app, name, cmd, options, callback) {
var portEnv = []; var portEnv = [];
for (let portName in app.portBindings) { for (let portName in app.portBindings) {
const hostPort = app.portBindings[portName]; const hostPort = app.portBindings[portName];
const portType = portName in manifest.tcpPorts ? 'tcp' : 'udp'; const portType = (manifest.tcpPorts && portName in manifest.tcpPorts) ? 'tcp' : 'udp';
const ports = portType == 'tcp' ? manifest.tcpPorts : manifest.udpPorts; const ports = portType == 'tcp' ? manifest.tcpPorts : manifest.udpPorts;
var containerPort = ports[portName].containerPort || hostPort; var containerPort = ports[portName].containerPort || hostPort;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment