Spring/spring-data-redis
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Make RedisConnectionProxy public.

Browse Source 
RedisConnectionProxy is now public so it can be used properly for Java proxy creation across different class loaders. Previously, creating a Java proxy from a different class loader failed because of visibility/access restrictions.

Closes #2016
This commit is contained in:
Mark Paluch
2021-03-24 09:33:02 +01:00
parent 3b499c9dc1
commit f002bea3b2
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/springframework/data/redis/core/RedisConnectionUtils.java
View File

@@ -626,7 +626,7 @@ public abstract class RedisConnectionUtils {
* @since 2.4.2
* @see RedisConnectionUtils#getTargetConnection(RedisConnection)
*/
interface RedisConnectionProxy extends RedisConnection, RawTargetAccess {
public interface RedisConnectionProxy extends RedisConnection, RawTargetAccess {
/**
* Return the target {@link RedisConnection} of this proxy.

5
src/test/java/org/springframework/data/redis/core/RedisConnectionUtilsUnitTests.java
View File

@@ -104,14 +104,15 @@ class RedisConnectionUtilsUnitTests {
assertThat(TransactionSynchronizationManager.hasResource(factoryMock)).isFalse();
}
@Test // DATAREDIS-891
@Test // DATAREDIS-891, GH-2016
void bindConnectionShouldBindConnectionToOngoingTransactionScope() {
TransactionTemplate template = new TransactionTemplate(new DummyTransactionManager());
template.executeWithoutResult(status -> {
assertThat(RedisConnectionUtils.bindConnection(factoryMock, true)).isNotNull();
assertThat(RedisConnectionUtils.bindConnection(factoryMock, true))
.isInstanceOf(RedisConnectionUtils.RedisConnectionProxy.class);
assertThat(TransactionSynchronizationManager.hasResource(factoryMock)).isTrue();
assertThat(RedisConnectionUtils.getConnection(factoryMock)).isNotNull();