在 AppNamespace 进行快速删除并重建的操作场景中，AppNamespaceServiceWithCache 类在执行缓存清理时存在缺陷，会导致误删缓存key的问题

[hezhaoye]

描述bug

在 AppNamespace 进行快速删除并重建的操作场景中，AppNamespaceServiceWithCache 类在执行缓存清理时存在缺陷，会导致误删缓存key的问题，导致format字段判断不准

复现

通过如下步骤可以复现：

1. 管理员在Apollo管理员后台执行删除AppNamespace操作
2. 用户立马在Apollo应用里面重新创建同名namespace（时间间隔低于1分钟）
3. AppNamespaceServiceWithCache类在扫描清理过期key是60秒执行1次，这样子导致的结果就是，scanNewAppNamespaces()是每秒执行会快速的扫描ID到cache中，而updateAndDeleteCache()是每60秒执行1次，当执行删除操作时，会找到旧ID对应的appId+appNamespace进而删除，导致cache数据没了。而新创建的appNamespace因为之前已经快速扫描过了，不会再扫描了，导致bug出现。（如果重建时间间隔大于1分钟就正常）

期望

在1分钟内重建appNamespace，缓存也能正常识别生效。

截图

[dosubot]

这个 bug 的根本原因是：在 AppNamespace 被删除后，如果在 1 分钟内用同名重新创建 namespace，缓存清理逻辑会把新创建的 namespace 也一并删除，导致拉取配置时判断不准。具体来说，handleDeletedAppNamespaces 方法会根据旧的 appId+namespaceName key 删除缓存，但不会校验缓存里的对象 ID 是否和被删除的 ID 一致，这样就可能误删了新创建的 namespace（同名但不同 ID）源码分析。

修复建议：在删除缓存时，除了用 key 查找，还要判断缓存对象的 ID 是否等于被删除的 ID，只有一致时才删除。类似的做法在 publicAppNamespaceCache 已经有了（用对象引用判断），但 appNamespaceCache 还没有。可以参考 AccessKeyServiceWithCache 的处理方式，删除时用 ID 校验，避免误删新对象相关实现。

如果需要更彻底，可以考虑同步 scan 和 update/delete 的执行间隔，或者在缓存更新时增加时间戳校验，防止类似的竞态问题。

这样可以保证即使在 1 分钟内重建同名 namespace，缓存也能正常识别和生效。

To reply, just mention @dosu.

---

Docs are dead. Just use Dosu.

[spaceluke]

#5483 看来是相似的问题

[wyxmttk]

我来试试处理这个问题