This is an automated email from the ASF dual-hosted git repository.
udo pushed a change to branch feature/GEODE-3705
in repository https://gitbox.apache.org/repos/asf/geode.git.
from 38e031b GEODE-3861: Marking test as Flaky
new 7b1efca Initial commit to to clean up Authentication + Authorzation
issues
new 0182197 Adding SecurityProcessor.java and returning
MessageExecutionContext.java
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../internal/cache/tier/sockets/AcceptorImpl.java | 119 +++++++++-----------
.../cache/tier/sockets/ClientProtocolService.java | 2 +
.../sockets/GenericProtocolServerConnection.java | 8 +-
.../tier/sockets/ClientProtocolMessageHandler.java | 17 +--
.../tier/sockets/MessageExecutionContext.java | 53 ++++++---
.../{protobuf => }/EncodingTypeTranslator.java | 3 +-
.../internal/protocol/{protobuf => }/Failure.java | 20 +++-
.../apache/geode/internal/protocol/Handshaker.java | 63 +++++++++++
.../protocol/{protobuf => }/OperationContext.java | 3 +-
.../internal/protocol/ProtobufCachePipeline.java | 34 ++----
.../internal/protocol/ProtobufLocatorPipeline.java | 6 +-
.../internal/protocol/ProtobufProtocolService.java | 61 +++-------
.../protocol/{protobuf => }/ProtocolErrorCode.java | 3 +-
.../internal/protocol/{protobuf => }/Result.java | 7 +-
.../internal/protocol/{protobuf => }/Success.java | 21 +++-
.../ClientProtocolHandshaker.java} | 20 ++--
.../protocol/operations/OperationHandler.java | 6 +-
.../protocol/protobuf/ProtobufOpsProcessor.java | 64 ++++++-----
.../protobuf/ProtobufSerializationService.java | 1 +
.../protocol/protobuf/ProtobufStreamProcessor.java | 27 +++--
.../operations/GetAllRequestOperationHandler.java | 20 ++--
.../GetAvailableServersOperationHandler.java | 12 +-
.../GetRegionNamesRequestOperationHandler.java | 12 +-
.../GetRegionRequestOperationHandler.java | 18 +--
.../operations/GetRequestOperationHandler.java | 28 ++---
.../operations/PutAllRequestOperationHandler.java | 20 ++--
.../operations/PutRequestOperationHandler.java | 22 ++--
.../operations/RemoveRequestOperationHandler.java | 22 ++--
.../HandshakerRequestOperationHandler.java | 54 +++++++++
.../AuthenticationRequestOperationHandler.java | 57 ++++++++++
.../security/InvalidConfigAuthenticator.java | 23 ++--
.../security/ProtobufShiroAuthenticator.java | 60 ++--------
.../protobuf/security/ProtobufShiroAuthorizer.java | 6 +-
.../AuthenticationSecurityProcessor.java} | 24 ++--
.../AuthorizationSecurityProcessor.java} | 24 ++--
.../NoAuthenticationSecurityProcessor.java} | 23 ++--
.../statistics/ProtobufClientStatisticsImpl.java | 15 ++-
.../utilities/ProtobufResponseUtilities.java | 4 +-
.../protobuf/utilities/ProtobufUtilities.java | 8 +-
.../registry/OperationContextRegistry.java | 12 +-
.../AuthenticationLookupService.java} | 31 +++---
.../{protobuf => }/security/Authenticator.java | 21 +---
.../AuthorizationLookupService.java} | 34 +++---
.../{protobuf => }/security/Authorizer.java | 4 +-
.../{protobuf => }/security/NoOpAuthenticator.java | 10 +-
.../{protobuf => }/security/NoOpAuthorizer.java | 4 +-
.../SecurityProcessor.java} | 20 +---
.../{protobuf => }/statistics/NoOpStatistics.java | 4 +-
.../ProtocolClientStatistics.java} | 8 +-
geode-protobuf/src/main/proto/clientProtocol.proto | 4 +-
.../Result.java => proto/handshake_API.proto} | 28 +++--
.../protocol/AuthenticationIntegrationTest.java | 17 ++-
.../protocol/AuthorizationIntegrationTest.java | 7 +-
.../protocol/ProtobufProtocolServiceJUnitTest.java | 6 +-
.../acceptance/CacheConnectionJUnitTest.java | 6 +-
.../acceptance/CacheOperationsJUnitTest.java | 5 +-
.../acceptance/LocatorConnectionDUnitTest.java | 6 +-
...TypeToSerializationTypeTranslatorJUnitTest.java | 1 +
.../internal/protocol/protobuf/HandshakerTest.java | 124 +++++++++++++++++++++
.../ProtobufShiroAuthenticatorJUnitTest.java | 42 ++++---
.../protobuf/ProtobufStreamProcessorTest.java | 10 +-
.../protobuf/ProtobufTestExecutionContext.java | 15 +--
.../protocol/protobuf/ProtobufTestUtilities.java | 60 ++++++++++
.../GetAllRequestOperationHandlerJUnitTest.java | 13 ++-
...tAvailableServersOperationHandlerJUnitTest.java | 6 +-
...egionNamesRequestOperationHandlerJUnitTest.java | 9 +-
.../GetRegionRequestOperationHandlerJUnitTest.java | 11 +-
.../GetRequestOperationHandlerJUnitTest.java | 19 ++--
.../PutAllRequestOperationHandlerJUnitTest.java | 11 +-
.../PutRequestOperationHandlerJUnitTest.java | 17 +--
.../RemoveRequestOperationHandlerJUnitTest.java | 17 +--
71 files changed, 951 insertions(+), 621 deletions(-)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/EncodingTypeTranslator.java (96%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/Failure.java (64%)
create mode 100644
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/Handshaker.java
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/OperationContext.java (96%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/ProtocolErrorCode.java (94%)
copy geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf
=> }/Result.java (83%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/Success.java (66%)
copy
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf/Result.java
=> handshaker/ClientProtocolHandshaker.java} (63%)
create mode 100644
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/protobuf/operations/handshaker/HandshakerRequestOperationHandler.java
create mode 100644
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/protobuf/operations/security/AuthenticationRequestOperationHandler.java
copy
geode-protobuf/src/{test/java/org/apache/geode/internal/protocol/protobuf/ProtobufTestExecutionContext.java
=>
main/java/org/apache/geode/internal/protocol/protobuf/security/processors/AuthenticationSecurityProcessor.java}
(53%)
copy
geode-protobuf/src/{test/java/org/apache/geode/internal/protocol/protobuf/ProtobufTestExecutionContext.java
=>
main/java/org/apache/geode/internal/protocol/protobuf/security/processors/AuthorizationSecurityProcessor.java}
(54%)
copy
geode-protobuf/src/{test/java/org/apache/geode/internal/protocol/protobuf/ProtobufTestExecutionContext.java
=>
main/java/org/apache/geode/internal/protocol/protobuf/security/processors/NoAuthenticationSecurityProcessor.java}
(58%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/registry/OperationContextRegistry.java (89%)
copy
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf/security/NoOpAuthenticator.java
=> security/AuthenticationLookupService.java} (51%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/security/Authenticator.java (63%)
copy
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf/security/NoOpAuthenticator.java
=> security/AuthorizationLookupService.java} (50%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/security/Authorizer.java (85%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/security/NoOpAuthenticator.java (76%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/security/NoOpAuthorizer.java (86%)
copy
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf/statistics/ProtobufClientStatistics.java
=> security/SecurityProcessor.java} (67%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf =>
}/statistics/NoOpStatistics.java (89%)
rename
geode-protobuf/src/main/java/org/apache/geode/internal/protocol/{protobuf/statistics/ProtobufClientStatistics.java
=> statistics/ProtocolClientStatistics.java} (85%)
rename
geode-protobuf/src/main/{java/org/apache/geode/internal/protocol/protobuf/Result.java
=> proto/handshake_API.proto} (66%)
create mode 100644
geode-protobuf/src/test/java/org/apache/geode/internal/protocol/protobuf/HandshakerTest.java
create mode 100644
geode-protobuf/src/test/java/org/apache/geode/internal/protocol/protobuf/ProtobufTestUtilities.java
--
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].
