Merge pull request #1776 from bosch-io/bugfix/preserve-exception-stat…

…us-code Preserve original error http status code
eclipse-ditto · Oct 16, 2023 · d75deef · d75deef
2 parents b9f957f + fe5675c
commit d75deef
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();