I'm in the weeds here of Gremlin Server, but I'm planning to add a new
method to the Authenticator interface that will replace the current:

public SaslNegotiator newSaslNegotiator();

the preferred new method would be:

public default SaslNegotiator newSaslNegotiator(final InetAddress
remoteAddress)

This allows the SASL implementation to have some extra information in
creating its instance by giving it the client IP address.  Since this
method will accept null, i can easily deprecate the old method and provide
a default implementation of the new one that calls the old one.  Then,
Gremlin Server can call the new one and everyone's stuff can still work
nicely - so this is not a breaking change, but does include deprecation.

If no objections in the next 72 hours (Sunday, November 29, 2015 at
6:45am), I'll assume lazy consensus and move forward with this for
3.1.1-incubating.

Reply via email to