Fix using blank? since that might not exist, and also allow for adding user controls even if the paged and ber_sort flags weren't set

Author: gwillcox-r7

lib/net/ldap/connection.rb

@@ -425,7 +425,7 @@ def search(args = nil)
         # this breaks when calling to_ber. (Can't force binary data to UTF-8)
         # we have to disable paging (even though server supports it) to get around this...
 
-        controls_temp = args.fetch(:controls, [])
+        user_controls = args.fetch(:controls, [])
         controls = []
         controls <<
           [
@@ -435,10 +435,10 @@ def search(args = nil)
             rfc2696_cookie.map(&:to_ber).to_ber_sequence.to_s.to_ber,
           ].to_ber_sequence if paged
         controls << ber_sort if ber_sort
-        if controls.empty?
+        if controls.empty? && user_controls.empty?
           controls = nil
         else
-          controls += controls_temp unless controls_temp.blank?
+          controls += user_controls
           controls = controls.to_ber_contextspecific(0)
         end