[JIRA] Commented: (SC-170) Expose LDAP pool blockWaitTime on VT pool strategy via connector config
Scott Cantor (JIRA)
noreply at shibboleth.net
Thu Feb 9 16:30:38 GMT 2012
[ https://issues.shibboleth.net/jira/browse/SC-170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13805#comment-13805 ]
Scott Cantor commented on SC-170:
---------------------------------
I see it throwing the exception now and the hang is fixed. The pool init failing does take the IdP out entirely, though, which is not what it does when the connector with no pooling just fails to connect to LDAP. I think those should be uniformly handled in the connector.
But it appears that the blockWaitTime may not be needed to address the hang?
> Expose LDAP pool blockWaitTime on VT pool strategy via connector config
> -----------------------------------------------------------------------
>
> Key: SC-170
> URL: https://issues.shibboleth.net/jira/browse/SC-170
> Project: Shibboleth Common - Java
> Issue Type: Improvement
> Components: Attribute Authority
> Affects Versions: 1.3.4
> Reporter: Scott Cantor
> Assignee: Daniel W Fisher
> Attachments: vt-ldap-3.3.5-SNAPSHOT-dist.tar.gz
>
>
> If you have a pool configured in the LDAP connector, and the LDAP server doesn't respond or there's a trust-related error preventing connections, the IdP fails to start up but also hangs inside the pool init method waiting for at least one connection to come back. There's a blockWaitTime setting, but it apparently defaults to indefinite wait, and it isn't exposed in our config.
> We should at least expose it via the Pool config schema that wraps the internal VT pool, and use a better default, or perhaps use different defaults based on whether a hard or soft pool is used?
> Daniel also mentioned changing the VT library to throw an exception out if the minimum number of connections can't be set up in the pool.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the commits
mailing list