You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@synapse.apache.org by "n-jay (via GitHub)" <gi...@apache.org> on 2023/02/16 13:01:59 UTC

[GitHub] [synapse] n-jay opened a new pull request, #79: Add environment variable injection for LoadBalance endpoints

n-jay opened a new pull request, #79:
URL: https://github.com/apache/synapse/pull/79

   ## Purpose
   Add environment variable injection capability to LoadBalance endpoints.
   
   ## Approach
   Amended `AddressEndpointFactory` classes to extract environment variables via `System.getenv()` based on `$SYSTEM` prefix presence in the Synapse config XML URI elements.
   
   ## Automation tests
    - Unit tests 
      Covers related code written in abstract class `EndpointFactory`.
    - Integration tests
      N/A
   
   ## Related PRs
   Builds on the work done on PR https://github.com/apache/synapse/pull/55
   
   ## Test environment
   - Java version 1.8.0_141
   - Ubuntu 20.04            


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
For additional commands, e-mail: dev-help@synapse.apache.org


[GitHub] [synapse] n-jay merged pull request #79: Add environment variable injection for LoadBalance endpoints

Posted by "n-jay (via GitHub)" <gi...@apache.org>.
n-jay merged PR #79:
URL: https://github.com/apache/synapse/pull/79


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
For additional commands, e-mail: dev-help@synapse.apache.org


[GitHub] [synapse] isudana commented on pull request #79: Add environment variable injection for LoadBalance endpoints

Posted by "isudana (via GitHub)" <gi...@apache.org>.
isudana commented on PR #79:
URL: https://github.com/apache/synapse/pull/79#issuecomment-1451958028

   This just adds a test case, isn't it? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
For additional commands, e-mail: dev-help@synapse.apache.org


[GitHub] [synapse] n-jay commented on pull request #79: Add environment variable injection for LoadBalance endpoints

Posted by "n-jay (via GitHub)" <gi...@apache.org>.
n-jay commented on PR #79:
URL: https://github.com/apache/synapse/pull/79#issuecomment-1452022077

   Yup. Just to indicate that variable injection works with load balancing for static endpoints


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
For additional commands, e-mail: dev-help@synapse.apache.org