You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@santuario.apache.org by "darkmoonka (via GitHub)" <gi...@apache.org> on 2023/07/31 11:46:36 UTC
[GitHub] [santuario-xml-security-java] darkmoonka opened a new pull request, #195: Make provider in JCEMapper.java Thread-safe
darkmoonka opened a new pull request, #195:
URL: https://github.com/apache/santuario-xml-security-java/pull/195
-Introduce a ThreadLocal provider configuration that can hold a thread specific default provider. This way concurrency on the default provider can be avoided.
Based on the discussion and points here: https://github.com/apache/santuario-xml-security-java/pull/184#issuecomment-1607276879, I opened this PR to make the default provider configurable per thread. Also this solution is backward compatible, so this should not break any existing integrations.
--
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@santuario.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [santuario-xml-security-java] coheigea commented on pull request #195: Make provider in JCEMapper.java Thread-safe
Posted by "coheigea (via GitHub)" <gi...@apache.org>.
coheigea commented on PR #195:
URL: https://github.com/apache/santuario-xml-security-java/pull/195#issuecomment-1659858330
@seanjmullan Can you review this please?
--
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@santuario.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [santuario-xml-security-java] coheigea merged pull request #195: Make provider in JCEMapper.java Thread-safe
Posted by "coheigea (via GitHub)" <gi...@apache.org>.
coheigea merged PR #195:
URL: https://github.com/apache/santuario-xml-security-java/pull/195
--
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@santuario.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [santuario-xml-security-java] darkmoonka commented on pull request #195: Make provider in JCEMapper.java Thread-safe
Posted by "darkmoonka (via GitHub)" <gi...@apache.org>.
darkmoonka commented on PR #195:
URL: https://github.com/apache/santuario-xml-security-java/pull/195#issuecomment-1689611946
Thank you guys @coheigea and @seanjmullan !
Do you have some rough ideas when the next release 3.0.3 is to be expected?
--
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@santuario.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Make provider in JCEMapper.java Thread-safe [santuario-xml-security-java]
Posted by "coheigea (via GitHub)" <gi...@apache.org>.
coheigea commented on PR #195:
URL: https://github.com/apache/santuario-xml-security-java/pull/195#issuecomment-1760886401
@darkmoonka A vote is being called today, expect a release next week.
--
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@santuario.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [santuario-xml-security-java] seanjmullan commented on pull request #195: Make provider in JCEMapper.java Thread-safe
Posted by "seanjmullan (via GitHub)" <gi...@apache.org>.
seanjmullan commented on PR #195:
URL: https://github.com/apache/santuario-xml-security-java/pull/195#issuecomment-1686398486
> @seanjmullan Can you review this please?
This fix looks fine to me.
--
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@santuario.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org