From 72d0ae85e523b39689c8318726d9d45720cb7070 Mon Sep 17 00:00:00 2001 From: Michael Hoennig Date: Tue, 10 Sep 2024 13:50:44 +0200 Subject: [PATCH] fix import with domain setup dns verification --- .../validators/HsDomainSetupHostingAssetValidator.java | 8 +++++--- .../hsadminng/hs/migration/ImportHostingAssets.java | 1 + 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/main/java/net/hostsharing/hsadminng/hs/hosting/asset/validators/HsDomainSetupHostingAssetValidator.java b/src/main/java/net/hostsharing/hsadminng/hs/hosting/asset/validators/HsDomainSetupHostingAssetValidator.java index 40530ad1..20fcbf69 100644 --- a/src/main/java/net/hostsharing/hsadminng/hs/hosting/asset/validators/HsDomainSetupHostingAssetValidator.java +++ b/src/main/java/net/hostsharing/hsadminng/hs/hosting/asset/validators/HsDomainSetupHostingAssetValidator.java @@ -26,9 +26,11 @@ class HsDomainSetupHostingAssetValidator extends HostingAssetEntityValidator { @Override public List validateEntity(final HsHostingAsset assetEntity) { - final var violations = // new ArrayList(); - super.validateEntity(assetEntity); - if (!violations.isEmpty()) { + final var violations = super.validateEntity(assetEntity); + if (!violations.isEmpty() || assetEntity.isLoaded()) { + // it makes no sense to do DNS-based validation + // if the entity is already persisted or + // if the identifier (domain name) or structure is already invalid return violations; } diff --git a/src/test/java/net/hostsharing/hsadminng/hs/migration/ImportHostingAssets.java b/src/test/java/net/hostsharing/hsadminng/hs/migration/ImportHostingAssets.java index 2f34ecee..d3ed3407 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/migration/ImportHostingAssets.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/migration/ImportHostingAssets.java @@ -1450,6 +1450,7 @@ public class ImportHostingAssets extends BaseOfficeDataImport { // nothing here )) .build(); + domainSetupAsset.markAsLoaded(); // to skip setup verification domainSetupsByName.put(domain_name, domainSetupAsset); domainSetupAssets.put(domain_id, domainSetupAsset); domainSetupAsset.setSubHostingAssets(new ArrayList<>()); -- 2.39.5