Re: Review Request 60375: GEODE-3117: fix Gateway authentication with legacy security

2017-06-29 Thread Jinmei Liao
> On June 26, 2017, 11:58 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java > > Line 82 (original), 82 (patched) > > > > > > since we already pass

Re: Review Request 60375: GEODE-3117: fix Gateway authentication with legacy security

2017-06-29 Thread Kirk Lund
> On June 26, 2017, 11:58 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java > > Line 82 (original), 82 (patched) > > > > > > since we already pass

Re: Review Request 60375: GEODE-3117: fix Gateway authentication with legacy security

2017-06-29 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60375/ --- (Updated June 29, 2017, 5:29 p.m.) Review request for geode, Emily Yeh, Jinmei

Re: Review Request 60375: GEODE-3117: fix Gateway authentication with legacy security

2017-06-29 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60375/#review179274 --- geode-wan/src/test/java/org/apache/geode/internal/cache/wan/misc/

Re: Review Request 60375: GEODE-3117: fix Gateway authentication with legacy security

2017-06-27 Thread Jinmei Liao
> On June 26, 2017, 11:58 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java > > Line 82 (original), 82 (patched) > > > > > > since we already pass

Re: Review Request 60375: GEODE-3117: fix Gateway authentication with legacy security

2017-06-27 Thread Kirk Lund
> On June 26, 2017, 11:58 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java > > Line 82 (original), 82 (patched) > > > > > > since we already pass

Re: Review Request 60375: GEODE-3117: fix Gateway authentication with legacy security

2017-06-26 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60375/#review178940 --- geode-core/src/main/java/org/apache/geode/cache/client/internal/C