Remove tax base payload & dont send stringifed tax calc result

saleor · Nov 15, 2024 · fee32ca · fee32ca
1 parent 5448145
commit fee32ca
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/apps/avatax/src/pages/api/webhooks/checkout-calculate-taxes.ts b/apps/avatax/src/pages/api/webhooks/checkout-calculate-taxes.ts
@@ -47,10 +47,6 @@ const handler = checkoutCalculateTaxesSyncWebhook.createHandler(async (req, res,
 
     subscriptionErrorChecker.checkPayload(payload);
 
-    logger.info("Tax base payload for checkout calculate taxes", {
-      payload: payload.taxBase,
-    });
-
     loggerContext.set(ObservabilityAttributes.CHANNEL_SLUG, ctx.payload.taxBase.channel.slug);
     loggerContext.set(ObservabilityAttributes.CHECKOUT_ID, ctx.payload.taxBase.sourceObject.id);
 

diff --git a/apps/avatax/src/pages/api/webhooks/order-calculate-taxes.ts b/apps/avatax/src/pages/api/webhooks/order-calculate-taxes.ts
@@ -207,8 +207,7 @@ const handler = orderCalculateTaxesSyncWebhook.createHandler(async (req, res, ct
       discountStrategy,
     );
 
-    // eslint-disable-next-line @saleor/saleor-app/logger-leak
-    logger.info("Taxes calculated", { calculatedTaxes: JSON.stringify(calculatedTaxes) });
+    logger.info("Taxes calculated", { calculatedTaxes: calculatedTaxes });
 
     ClientLogStoreRequest.create({
       level: "info",