Skip to content

Conversation

@msm1992
Copy link
Contributor

@msm1992 msm1992 commented Oct 15, 2025

Adds config changes for wso2/wso2-synapse#2409

Todo

  • Need to bump synapse version once above synapse PR is merged.

Copy link

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

This PR adds Jinja2 template configuration support for a new synapse property that controls retry message building behavior in backend OAuth flows. The changes enable conditional rendering of the synapse.endpoint.http.oauth.token.retry.message.building.disabled property based on the presence of a new configuration variable.

  • Adds conditional template block for disable_retry_message_building configuration
  • Updates synapse.properties.j2 templates across all deployment modules
  • Supports the corresponding synapse PR #2409

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
traffic-manager/modules/distribution/product/src/main/resources/conf/templates/repository/conf/synapse.properties.j2 Adds conditional rendering of retry message building disabled property for traffic manager module
gateway/modules/distribution/product/src/main/resources/conf/templates/repository/conf/synapse.properties.j2 Adds conditional rendering of retry message building disabled property for gateway module
api-control-plane/modules/distribution/product/src/main/resources/conf/templates/repository/conf/synapse.properties.j2 Adds conditional rendering of retry message building disabled property for API control plane module
all-in-one-apim/modules/distribution/product/src/main/resources/conf/templates/repository/conf/synapse.properties.j2 Adds conditional rendering of retry message building disabled property for all-in-one APIM module

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@msm1992 msm1992 force-pushed the master branch 2 times, most recently from c04859c to e63a7e1 Compare October 20, 2025 05:04
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.

1 participant