Skip to content

Add nginx location block for /api/v1/webhooks/*/sync with extended timeout for docker compose deployments#2066

Merged
MarceloRGonc merged 9 commits intomainfrom
cezudas/OPS-3628
Mar 5, 2026
Merged

Add nginx location block for /api/v1/webhooks/*/sync with extended timeout for docker compose deployments#2066
MarceloRGonc merged 9 commits intomainfrom
cezudas/OPS-3628

Conversation

@cezudas
Copy link
Contributor

@cezudas cezudas commented Mar 5, 2026

Fixes OPS-3854.

@cezudas cezudas requested a review from Copilot March 5, 2026 09:44
@linear
Copy link

linear bot commented Mar 5, 2026

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a dedicated NGINX routing rule in the docker-compose gateway config to support long-running webhook “sync” requests without timing out, aligning with OPS-3628’s deployment needs.

Changes:

  • Introduces a regex location for /api/v1/webhooks/*/sync.
  • Applies extended proxy and send/read/connect timeouts (600s) for that endpoint in docker-compose deployments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@cezudas cezudas force-pushed the cezudas/OPS-3628 branch from c66f85a to f6be396 Compare March 5, 2026 10:08
@cezudas cezudas requested a review from MarceloRGonc March 5, 2026 11:29
@cezudas cezudas marked this pull request as ready for review March 5, 2026 11:29
@cezudas cezudas requested a review from maor-rozenfeld as a code owner March 5, 2026 11:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 5, 2026

@MarceloRGonc MarceloRGonc merged commit c49dfae into main Mar 5, 2026
25 checks passed
@MarceloRGonc MarceloRGonc deleted the cezudas/OPS-3628 branch March 5, 2026 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants