mhansonp opened a new pull request #6912: URL: https://github.com/apache/geode/pull/6912
Basically we have two problems that are being dealt with here both in the context of MultiUserAuth. 1) When we register a new DataSerializer, we should be supplying credentials to enable the registration process to talk to the server. 2) When we get notified of a new endpoint being up, DataSerializerRecoveryTask is called. In this class we don't have a copy of the auth information which causes a warning and crash. Part 1 was fixed by plumbing the credentials down to the point of the registration with the server. Part 2 has the problem the there are really two levels of authentication going on. Cluster level and user level. The addition of DataSerializers is a cluster level operations. Adding new data is user level. As a first cut, I have the DataSerializerRecoveryTask reach into the system settings and grab the credentials it needs. I am not sure if this is satisfactory. Hence this is not a pull request per se. Just a draft... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
