Author: Andy Green <andy@warmcat.com>
Date: Tue Mar 16 19:37:23 2021 +0000

align with set_metadata requiring result check


diff --git a/assets/index.html b/assets/index.html
index e37102f..0053d50 100644
--- a/assets/index.html
+++ b/assets/index.html
@@ -9,11 +9,11 @@
  </head>
 	<body>
 		<div class="summary nailed">
+		<table><tr><td class="topregion">
 			<div class="logo">
-				<img class="logo" src="sai.svg">
+				<img class="logo" src="sai.svg"><br>
 				<a href="https://warmcat.com/git/sai">Sai git</a>
-			</div>
-
+			<div>
 			<div id="login" class="login">
 				<div id="creds" class="creds hide">
 					 <form class="login">
@@ -30,10 +30,13 @@
 					<input class="crin" type="button" id="logout" value="Logout">
 				</div>
 			</div>
-
-			<div class="builder">
+			</td><td class="topregion">
+			
+									<div class="builder">
 				<div id="sai_builders"></div>
 			</div>
+			</td></tr></table>
+
 		</div>	
 		<div id="sai_sticky" class="sticky"></div>
 		<div id="p2" class="undernailed">
diff --git a/assets/sai.css b/assets/sai.css
index c8b3ab8..039b69e 100644
--- a/assets/sai.css
+++ b/assets/sai.css
@@ -203,9 +203,9 @@ div.taskinfo {
 	background: #fff;
 	z-index: 2002;
 	border-style: solid;
-	border-width: 16px 16px 0px 16px;
+	border-width: 3px 16px 0px 16px;
 	border-color: #fff;
-	margin-top: -16px;
+	margin-top: -3px;
 }
 
 table.scrollogs {
@@ -222,7 +222,6 @@ div.taskstate {
 }
 
 div.nailed {
-        position:fixed;
         background: #fff;
         width:auto;
         z-index:1001;
@@ -443,7 +442,7 @@ form.login {
 }
 
 div.login {
-	display: block;
+	display: inline-block;
 	vertical-align: middle;
 	font-size: 7pt;
 	color:#3d9970;
@@ -451,8 +450,6 @@ div.login {
 	margin: 0px 16px 0px 16px;
 	padding: 2px 3px;
 	border-radius:5px;
-	float: left;
-	width: auto;
 }
 
 
@@ -537,21 +534,24 @@ img.bsvg {
 }
 
 div.builder {
-	display: block;
-	position: absolute;
-	top: 150px;
+	display: inline-block;
 }
 
 div.sticky {
-	margin-left: 180px;
+	float: left;
+	position: absolute;
 }
 
+td.topregion {
+	vertical-align: top
+}
 
 div.logo {
+	display: inline-block;
 	margin: 3px;
 	margin-right: 4px;
 	padding: 2px;
-	vertical-align: top;
+	vertical-align: middle;
 	z-index: 1002;
 	width: 164px;
 	height: auto;
diff --git a/src/builder/b-comms.c b/src/builder/b-comms.c
index c38fc52..e3dc155 100644
--- a/src/builder/b-comms.c
+++ b/src/builder/b-comms.c
@@ -567,7 +567,8 @@ saib_m_state(void *userobj, void *sh, lws_ss_constate_t state,
 		((char *)spm->url)[ctx->npos] = '\0';
 
 		lwsl_notice("%s: binding ss to %s\n", __func__, spm->url);
-		lws_ss_set_metadata(spm->ss, "url", spm->url, strlen(spm->url));
+		if (lws_ss_set_metadata(spm->ss, "url", spm->url, strlen(spm->url)))
+			lwsl_warn("%s: unable to set metadata\n", __func__);
 
 		pq = spm->url;
 		while (*pq && (pq[0] != '/' || pq[1] != '/'))
diff --git a/src/builder/b-task.c b/src/builder/b-task.c
index d753427..a629084 100644
--- a/src/builder/b-task.c
+++ b/src/builder/b-task.c
@@ -266,7 +266,8 @@ artifact_glob_cb(void *data, const char *path)
 	 * as meaning the bulk data follows.
 	 */
 
-	lws_ss_set_metadata(h, "url", ns->spm->url, strlen(ns->spm->url));
+	if (lws_ss_set_metadata(h, "url", ns->spm->url, strlen(ns->spm->url)))
+		lwsl_warn("%s: unable to set metadata\n", __func__);
 	lws_ss_client_connect(h);
 
 	return 0;
diff --git a/src/common/ss-client-logproxy.c b/src/common/ss-client-logproxy.c
index b880163..36473e0 100644
--- a/src/common/ss-client-logproxy.c
+++ b/src/common/ss-client-logproxy.c
@@ -65,7 +65,8 @@ saicom_lp_ss_from_env(struct lws_context *context, const char *env_name)
 		return NULL;
 	}
 
-	lws_ss_set_metadata(h, "sockpath", e, strlen(e));
+	if (lws_ss_set_metadata(h, "sockpath", e, strlen(e)))
+		lwsl_warn("%s: metadata set failed\n", __func__);
 	lws_ss_client_connect(h);
 
 	return h;
diff --git a/src/resource/r-comms.c b/src/resource/r-comms.c
index 8e5dbca..2d0faac 100644
--- a/src/resource/r-comms.c
+++ b/src/resource/r-comms.c
@@ -130,7 +130,8 @@ sair_ss_from_env(struct lws_context *context, const char *env_name)
 		return NULL;
 	}
 
-	lws_ss_set_metadata(h, "sockpath", e, strlen(e));
+	if (lws_ss_set_metadata(h, "sockpath", e, strlen(e)))
+		lwsl_err("%s: unable to set metadata\n", __func__);
 	lws_ss_client_connect(h);
 
 	return h;
diff --git a/src/web/w-comms.c b/src/web/w-comms.c
index 2dc4511..e42ea09 100644
--- a/src/web/w-comms.c
+++ b/src/web/w-comms.c
@@ -510,8 +510,9 @@ callback_ws(struct lws *wsi, enum lws_callback_reasons reason, void *user,
 			return 1;
 		}
 
-		lws_ss_set_metadata(vhd->h_ss_websrv, "sockpath",
-				    "@com.warmcat.sai-websrv", 23);
+		if (lws_ss_set_metadata(vhd->h_ss_websrv, "sockpath",
+				    "@com.warmcat.sai-websrv", 23))
+			lwsl_warn("%s: unable to set metadata\n", __func__);
 		lws_ss_client_connect(vhd->h_ss_websrv);
 
 		break;