From 79c052101d800928a63fb43ad715fa32d3bb66ae Mon Sep 17 00:00:00 2001
From: Hemal Shah <hemal@gradientedge.com>
Date: Wed, 24 Sep 2025 11:19:16 +0100
Subject: [PATCH] feat(azure-apim): update apim cache conditions when bypassing
 cache

---
 src/lib/azure/services/api-management/main.ts | 37 +++++++++----------
 1 file changed, 18 insertions(+), 19 deletions(-)

diff --git a/src/lib/azure/services/api-management/main.ts b/src/lib/azure/services/api-management/main.ts
index 578070ea..463a6ad7 100644
--- a/src/lib/azure/services/api-management/main.ts
+++ b/src/lib/azure/services/api-management/main.ts
@@ -273,28 +273,27 @@ export class AzureApiManagementManager {
                           </when>
                       </choose>
                   </when>
+                  <when condition="@((string)context.Variables["bypassCache"] == "true")">
+                      <cache-remove-value key="@((string)context.Variables["customCacheKey"])" />
+                  </when>
               </choose>`
-          cacheSetOutboundPolicy = `<choose>
-                  <when condition="@((string)context.Variables["bypassCache"] != "true")">
-                      <!-- Store the response body in cache -->
-                      <choose>
-                          <when condition="@(context.Response.StatusCode == 200)">
-                              <cache-store-value key="@((string)context.Variables["customCacheKey"])" value="@(context.Response.Body.As<string>(preserveContent: true))" duration="${operation.caching.ttlInSecs ?? 900}" />
-                              <!-- Add cache status header -->
-                              <set-header name="X-Apim-Cache-Status" exists-action="override">
-                                  <value>MISS</value>
-                              </set-header>
-                          </when>
-                      </choose>
-                      <!-- Add debug headers -->
-                      <set-header name="X-Apim-Cache-Key" exists-action="override">
-                          <value>@((string)context.Variables["customCacheKey"])</value>
-                      </set-header>
-                      <set-header name="X-Apim-API-Name" exists-action="override">
-                          <value>@(context.Api.Name)</value>
+          cacheSetOutboundPolicy = `<!-- Store the response body in cache -->
+              <choose>
+                  <when condition="@(context.Response.StatusCode == 200)">
+                      <cache-store-value key="@((string)context.Variables["customCacheKey"])" value="@(context.Response.Body.As<string>(preserveContent: true))" duration="${operation.caching.ttlInSecs ?? 900}" />
+                      <!-- Add cache status header -->
+                      <set-header name="X-Apim-Cache-Status" exists-action="override">
+                          <value>MISS</value>
                       </set-header>
                   </when>
-              </choose>`
+              </choose>
+              <!-- Add debug headers -->
+              <set-header name="X-Apim-Cache-Key" exists-action="override">
+                  <value>@((string)context.Variables["customCacheKey"])</value>
+              </set-header>
+              <set-header name="X-Apim-API-Name" exists-action="override">
+                  <value>@(context.Api.Name)</value>
+              </set-header>`
         }
 
         if (operation.caching.enableCacheInvalidation) {