From eeb63613f28ebc1207571fbcc00eeb417d3b06c1 Mon Sep 17 00:00:00 2001 From: XiaoSeS <87064762+XiaoSeS@users.noreply.github.com> Date: Fri, 28 Aug 2026 16:42:26 +0800 Subject: [PATCH] fix(namespace): enable personal provisioning by default --- docs/2026-08-13-personal-namespace-provisioning.md | 4 ++-- server/skillhub-app/src/main/resources/application.yml | 2 +- .../namespace/PersonalNamespaceProvisioningProperties.java | 2 +- .../namespace/PersonalNamespaceProvisioningServiceTest.java | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/docs/2026-08-13-personal-namespace-provisioning.md b/docs/2026-08-13-personal-namespace-provisioning.md index b61b74ff..fe67433b 100644 --- a/docs/2026-08-13-personal-namespace-provisioning.md +++ b/docs/2026-08-13-personal-namespace-provisioning.md @@ -75,9 +75,9 @@ slug 模板渲染后按 `SlugValidator` 的规则归一化:转小写、 | 位置 | 项 | 默认 | |------|-----|------| | `application.yml` | `skillhub.namespace.personal-provisioning.enabled` | `false` | -| 配置文件/环境变量 | 启用开关 | `false` | +| 配置文件/环境变量 | 启用开关 | `true` | -**默认关闭**:升级不应该让现有部署突然开始建命名空间。 +默认只对新激活账号生效,不回填已有账号;如需关闭可设置环境变量。 模板刻意**不放在 `application.yml`**:它们含 `${...}`, Spring 会当成属性占位符去解析(Boot 3.2 / Framework 6.1 尚不支持转义 `\${`)。 diff --git a/server/skillhub-app/src/main/resources/application.yml b/server/skillhub-app/src/main/resources/application.yml index f851441e..c6a57def 100644 --- a/server/skillhub-app/src/main/resources/application.yml +++ b/server/skillhub-app/src/main/resources/application.yml @@ -120,7 +120,7 @@ skillhub: # Whether a newly activated account gets a namespace of its own. # Slug and display-name templates use safe code defaults in PersonalNamespaceProvisioningProperties. personal-provisioning: - enabled: ${SKILLHUB_NAMESPACE_PERSONAL_PROVISIONING_ENABLED:false} + enabled: ${SKILLHUB_NAMESPACE_PERSONAL_PROVISIONING_ENABLED:true} public: base-url: ${SKILLHUB_PUBLIC_BASE_URL:} access-policy: diff --git a/server/skillhub-domain/src/main/java/com/iflytek/skillhub/domain/namespace/PersonalNamespaceProvisioningProperties.java b/server/skillhub-domain/src/main/java/com/iflytek/skillhub/domain/namespace/PersonalNamespaceProvisioningProperties.java index 6d7c39ff..6c8f536f 100644 --- a/server/skillhub-domain/src/main/java/com/iflytek/skillhub/domain/namespace/PersonalNamespaceProvisioningProperties.java +++ b/server/skillhub-domain/src/main/java/com/iflytek/skillhub/domain/namespace/PersonalNamespaceProvisioningProperties.java @@ -15,7 +15,7 @@ public class PersonalNamespaceProvisioningProperties { /** * Off by default: existing deployments must not start creating namespaces after an upgrade. */ - private boolean enabled = false; + private boolean enabled = true; /** * Templates remain code defaults because Spring treats {@code ${...}} in YAML as property diff --git a/server/skillhub-domain/src/test/java/com/iflytek/skillhub/domain/namespace/PersonalNamespaceProvisioningServiceTest.java b/server/skillhub-domain/src/test/java/com/iflytek/skillhub/domain/namespace/PersonalNamespaceProvisioningServiceTest.java index fc61b617..d2066061 100644 --- a/server/skillhub-domain/src/test/java/com/iflytek/skillhub/domain/namespace/PersonalNamespaceProvisioningServiceTest.java +++ b/server/skillhub-domain/src/test/java/com/iflytek/skillhub/domain/namespace/PersonalNamespaceProvisioningServiceTest.java @@ -89,8 +89,8 @@ class PersonalNamespaceProvisioningServiceTest { } @Test - void defaultsAreDisabledSoUpgradesDoNotStartCreatingNamespaces() { - assertEquals(false, new PersonalNamespaceProvisioningProperties().isEnabled()); + void defaultsAreEnabledForNewAccountProvisioning() { + assertEquals(true, new PersonalNamespaceProvisioningProperties().isEnabled()); } @Test