Skip to content

Commit f16b4c1

Browse files
committed
Don't store username as label, because it may contain invalid data
1 parent f5d4acd commit f16b4c1

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

src/main/java/eu/openanalytics/containerproxy/backend/AbstractContainerBackend.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,6 @@ public abstract class AbstractContainerBackend implements IContainerBackend {
7171
protected static final String ENV_VAR_REALM_ID = "SHINYPROXY_REALM_ID";
7272

7373
protected static final String LABEL_PROXY_ID = "openanalytics.eu/sp-proxy-id";
74-
protected static final String LABEL_USER_ID = "openanalytics.eu/sp-user-id";
7574
protected static final String LABEL_PROXY_SPEC_ID = "openanalytics.eu/sp-spec-id";
7675
protected static final String LABEL_STARTUP_TIMESTAMP = "openanalytics.eu/sp-proxy-startup-timestamp";
7776

@@ -134,7 +133,6 @@ protected void doStartProxy(Proxy proxy) throws Exception {
134133

135134
// add labels need for App Recovery and maintenance
136135
spec.addLabel(LABEL_PROXY_ID, proxy.getId());
137-
spec.addLabel(LABEL_USER_ID, proxy.getUserId());
138136
spec.addLabel(LABEL_PROXY_SPEC_ID, proxy.getSpec().getId());
139137
spec.addLabel(LABEL_STARTUP_TIMESTAMP, String.valueOf(proxy.getStartupTimestamp()));
140138

@@ -144,7 +142,6 @@ protected void doStartProxy(Proxy proxy) throws Exception {
144142

145143
// remove labels needed for App Recovery since they do not really belong to the spec
146144
spec.removeLabel(LABEL_PROXY_ID);
147-
spec.removeLabel(LABEL_USER_ID);
148145
spec.removeLabel(LABEL_PROXY_SPEC_ID);
149146
spec.removeLabel(LABEL_STARTUP_TIMESTAMP);
150147

0 commit comments

Comments
 (0)