Description:
Need to add Http connection pooling for PublishToAnalyticsFunction, to improve performance in high load.
Need to have HTTP keep-alive.
Need to have HTTP idle connection eviction.

Idle timeout needs to be configurable.

Please refer http://www.baeldung.com/httpclient-connection-management

Affected Product Version:
5.6.0

OS, DB, other environment details and versions:

Steps to reproduce:

Description:
When I was working on wso2/product-is#6012, I had to upgrade the POM file version reference of carbon-identity-framework from v5.12.60 to the latest, in order to use the UserSessionManagementService. This results in a test failure when building the product. The reason for the failure is as follows.

• Class: org.wso2.carbon.identity.conditional.auth.functions.test.utils.sequence.JsSequenceHandlerAbstractTest
• Method: initialize()
• Line:

  identity-conditional-auth-functions/components/org.wso2.carbon.identity.conditional.auth.functions.test.utils/src/main/java/org/wso2/carbon/identity/conditional/auth/functions/test/utils/sequence/JsSequenceHandlerAbstractTest.java

  Line 70 in 0235977

  ((FileChannel) targetChannel).transferFrom(inputChannel, 0, Short.MAX_VALUE);

• Reason: The 3rd argument that gets passed to the transferFrom() is the count variable that indicates the count to be copied from the input stream. At the moment, we pass the value of Short.MAX_VALUE constant. Since the size of the channel exceeds the value of Short.MAX_VALUE constant, it only copies over half of the identity.xml file. This results in a test failure while throwing the exception org.apache.axiom.om.OMException: com.ctc.wstx.exc.WstxEOFException: Unexpected end of input block in start tag.

Affected Product Version: 0.1.35-SNAPSHOT

Steps to reproduce:
Upgrade POM file version reference of carbon-identity-framework to the latest and run $ mvn clean install

Description:
In order to use session based functions, specifically ones that use SessionValidationUtil, it is required that properties are set in identity.xml. The properties that need to be set are described here. For example, SessionBasedValidation.TableSearchURL.

The issue is that there is no relevant config mapping listed in the config-mapping repo. Because of this, when SessionValidationUtil attempts to get any "SessionBasedValidation" config properties they are null.

Suggested Labels:

Suggested Assignees:

Affected Product Version:
Identity Server 5.9.0 and later

OS, DB, other environment details and versions:
5.11.0 running in Docker

Steps to reproduce:

• Add the session functions JAR to the dropins folder
• Invoke a function, getSessionData for instance, in the Script Based Adaptive Authentication console
• NPE is thrown because the URL returned from the properties is null

Snippet of the stacktrace:

java.lang.NullPointerException
	at java.base/java.net.URI$Parser.parse(URI.java:3104)
	at java.base/java.net.URI.<init>(URI.java:600)
	at java.base/java.net.URI.create(URI.java:881)
	at org.apache.http.client.methods.HttpPost.<init>(HttpPost.java:76)
	at org.wso2.carbon.identity.conditional.auth.functions.session.util.SessionValidationUtil.createHttpRequest(SessionValidationUtil.java:158)
	at org.wso2.carbon.identity.conditional.auth.functions.session.util.SessionValidationUtil.getSessionDetails(SessionValidationUtil.java:69)
	at org.wso2.carbon.identity.conditional.auth.functions.session.function.GetSessionDataFunction.getData(GetSessionDataFunction.java:63)

Related Issues: