-
Notifications
You must be signed in to change notification settings - Fork 539
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add timeout server configurations to action execution #5935
Add timeout server configurations to action execution #5935
Conversation
Let's add a configuration the the http connection pool as well |
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
...tity.core.server.feature/resources/org.wso2.carbon.identity.core.server.feature.default.json
Outdated
Show resolved
Hide resolved
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5935 +/- ##
============================================
+ Coverage 21.07% 21.11% +0.04%
- Complexity 5626 5642 +16
============================================
Files 1561 1561
Lines 99202 99234 +32
Branches 15189 15185 -4
============================================
+ Hits 20904 20953 +49
+ Misses 75277 75257 -20
- Partials 3021 3024 +3 ☔ View full report in Codecov by Sentry. |
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
ba12141
to
af8cec3
Compare
af8cec3
to
5f6e873
Compare
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
...ution/src/main/java/org/wso2/carbon/identity/action/execution/util/ActionExecutorConfig.java
Outdated
Show resolved
Hide resolved
c69ecd8
to
d3435de
Compare
Proposed changes in this pull request
This PR adds http client related server configurations to action execution and updates unit tests in ActionExecutorConfigTest, APIClientTest and ActionExecutorServiceImpl classes. wso2/product-is#21119
When should this PR be merged
Soon
Follow up actions
N/A