🚨 SASL: add_authenticator warns on reassignment (🚧 ???)

The question is: do we even want to this?

Author: nevans

lib/net/imap/sasl/authenticators.rb

@@ -68,7 +68,7 @@ def names; @authenticators.keys end
     # When only a single argument is given, the authenticator class will be
     # lazily loaded from <tt>Net::IMAP::SASL::#{name}Authenticator</tt> (case is
     # preserved and non-alphanumeric characters are removed..
-    def add_authenticator(name, authenticator = nil)
+    def add_authenticator(name, authenticator = nil, warn_overwrite: true)
       authenticator ||= begin
         class_name = "#{name.gsub(/[^a-zA-Z0-9]/, "")}Authenticator".to_sym
         auth_class = nil
@@ -78,6 +78,11 @@ def add_authenticator(name, authenticator = nil)
         }
       end
       key = Authenticators.normalize_name(name)
+      if warn_overwrite && (original = @authenticators[key])
+        warn("%p: replacing existing %p authenticator: %p" % [
+          self, key, original
+        ], uplevel: 1)
+      end
       @authenticators[key] = authenticator
     end