Apologies for this if I already agreed to this name, but as I was reading this patch, I found myself confused what amp_server was and if it was different from amqp_host.  To me, they seemed like they were probably the same thing, so I thought we had duplication.  Then I scrolled back up here to see what the default value was and realized you were setting the type of amqp server with this.  What would you think of changing this param name to 'amqp_type'?