preserve httpStatus for other reason on faild create of thing

Signed-off-by: Aleksandar Stanchev <aleksandar.stanchev@bosch.com>
eclipse-ditto · Oct 16, 2023 · fe5675c · fe5675c
1 parent 670fb14
commit fe5675c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/...rg/eclipse/ditto/things/model/signals/commands/exceptions/ThingNotCreatableException.java b/...rg/eclipse/ditto/things/model/signals/commands/exceptions/ThingNotCreatableException.java
@@ -113,8 +113,9 @@ public static Builder newBuilderForPolicyExisting(final ThingId thingId, final P
      * @param reason the reason why the implicit policy creation failed.
      * @return the builder.
      */
-    public static Builder newBuilderForOtherReason(final ThingId thingId, final PolicyId policyId, final String reason) {
-        return new Builder(thingId, policyId, reason);
+    public static Builder newBuilderForOtherReason(final HttpStatus httpStatus, final ThingId thingId, final PolicyId policyId, final String reason) {
+        return new Builder(thingId, policyId, reason)
+                .httpStatus(httpStatus);
     }
 
     /**

diff --git a/...ervice/src/main/java/org/eclipse/ditto/things/service/enforcement/ThingEnforcerActor.java b/...ervice/src/main/java/org/eclipse/ditto/things/service/enforcement/ThingEnforcerActor.java
@@ -425,7 +425,7 @@ private ThingNotCreatableException reportInitialPolicyCreationFailure(final Poli
                     .dittoHeaders(command.getDittoHeaders())
                     .build();
         } else {
-            return ThingNotCreatableException.newBuilderForOtherReason(command.getEntityId(), policyId,
+            return ThingNotCreatableException.newBuilderForOtherReason(policyException.getHttpStatus(), command.getEntityId(), policyId,
                             policyException.getMessage())
                     .dittoHeaders(command.getDittoHeaders())
                     .build();