d538d0dc7d
## Description Updated the computeConsolidatedAPIResponseEtag method to return weak ETags instead of strong ETags to ensure compatibility with nginx when gzip is enabled. This change includes adding a prefix "W/" to the ETag value and includes references to related issues and documentation. Ref: https://github.com/kubernetes/ingress-nginx/issues/1390 Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.All" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/13069342399> > Commit: b09d978d0123359c20a16d02445f97d9b9102d83 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=13069342399&attempt=3" target="_blank">Cypress dashboard</a>. > Tags: `@tag.All` > Spec: > <hr>Fri, 31 Jan 2025 10:40:19 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Configuration Changes** - Modified NGINX configuration to potentially enable gzip compression for API responses - Updated ETag generation to use weak ETag format for better compatibility - Removed specific route handling for consolidated API endpoint in Caddy configuration <!-- end of auto-generated comment: release notes by coderabbit.ai --> |
||
|---|---|---|
| .. | ||
| appsmith | ||
| bin | ||