----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56459/#review164840 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/management/internal/web/shell/AbstractHttpOperationInvoker.java (line 82) <https://reviews.apache.org/r/56459/#comment236648> Is this change necessary? - Jinmei Liao On Feb. 9, 2017, 12:21 a.m., Kevin Duling wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56459/ > ----------------------------------------------------------- > > (Updated Feb. 9, 2017, 12:21 a.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/main/java/org/apache/geode/management/internal/web/shell/AbstractHttpOperationInvoker.java > fa052489772e3e03eb865d17dbbcb7e227813c42 > > 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 running > > > Thanks, > > Kevin Duling > >
