----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56459/#review165019 -----------------------------------------------------------
Ship it! Ship It! - Kirk Lund On Feb. 9, 2017, 8:50 p.m., Kevin Duling wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56459/ > ----------------------------------------------------------- > > (Updated Feb. 9, 2017, 8:50 p.m.) > > > Review request for geode, Jinmei Liao, Jared Stewart, and Kirk Lund. > > > Repository: geode > > > Description > ------- > > GEODE-2247: GFSH connect over HTTP without credentials should fail earlier > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/internal/security/IntegratedSecurityService.java > 6507295cd28b5f45f4de164c86c176f467a20f6e > > geode-core/src/main/java/org/apache/geode/internal/security/SecurityService.java > 727a1ce0c1c99edd5cd7299ddfbbcda4ca9a664f > > geode-core/src/main/java/org/apache/geode/management/internal/web/controllers/support/LoginHandlerInterceptor.java > 79c8c2717b3523837f01939cf67b3b38169c856b > > geode-core/src/test/java/org/apache/geode/distributed/internal/membership/gms/auth/AbstractGMSAuthenticatorTestCase.java > 65c38ad261ba759d5022fad4c1eb20249e71d69d > > geode-core/src/test/java/org/apache/geode/management/internal/security/IntegratedSecurityServiceWithIniFileJUnitTest.java > a300880d1d4a221d03c480df94b5b652175814f7 > > geode-web-api/src/main/java/org/apache/geode/rest/internal/web/security/GeodeAuthenticationProvider.java > c8b5cf0827032ae3148bfdeb1777fb8fc3b2d85e > > Diff: https://reviews.apache.org/r/56459/diff/ > > > Testing > ------- > > Log of UX on https://issues.apache.org/jira/browse/GEODE-2247 > precheckin successful > > > Thanks, > > Kevin Duling > >
