Fix #34445: remove mappings for failed proxy

LEDfan · LEDfan · commit 007b523df0a4 · 2025-01-20T15:11:11.000+01:00
diff --git a/src/main/java/eu/openanalytics/containerproxy/model/store/redis/RedisProxyStore.java b/src/main/java/eu/openanalytics/containerproxy/model/store/redis/RedisProxyStore.java
@@ -85,7 +85,8 @@ public void addProxy(Proxy proxy) {
     public void removeProxy(Proxy proxy) {
         logger.debug("Remove proxy {}", proxy.getId());
         ops.delete(redisKey, proxy.getId());
-        updateMappings(proxy.getId(), proxy);
+        logger.debug("Redis: remove mappings (event) {}", proxy.getId());
+        mappingManager.removeMappings(proxy.getId());
         userProxyOps.remove(userProxyRedisKey + proxy.getUserId(), proxy.getId());
     }
 

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,8 @@ public void addProxy(Proxy proxy) {`
`85`	`85`	`public void removeProxy(Proxy proxy) {`
`86`	`86`	`logger.debug("Remove proxy {}", proxy.getId());`
`87`	`87`	`ops.delete(redisKey, proxy.getId());`
`88`		`- updateMappings(proxy.getId(), proxy);`
	`88`	`+ logger.debug("Redis: remove mappings (event) {}", proxy.getId());`
	`89`	`+ mappingManager.removeMappings(proxy.getId());`
`89`	`90`	`userProxyOps.remove(userProxyRedisKey + proxy.getUserId(), proxy.getId());`
`90`	`91`	`}`
`91`	`92`