From eaba5696fdecbbf0970aec0717ecd6dca9468f68 Mon Sep 17 00:00:00 2001
From: Sage Weil <sage@redhat.com>
Date: Wed, 6 Feb 2019 17:43:42 -0600
Subject: [PATCH] msg/async: let client specify preferred order of modes

The server side has an allowed list, while the client has an ordered list
in order of preference.

Note that some of the options are used as both (e.g., cluster_modes) as they
are used at both connecting and accepting ends of the connection.

Signed-off-by: Sage Weil <sage@redhat.com>
---
 src/common/options.cc       | 4 ++--
 src/msg/async/ProtocolV2.cc | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/common/options.cc b/src/common/options.cc
index 1449abf9d51..5fd88dacacf 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -850,7 +850,7 @@ std::vector<Option> get_global_options() {
 
     Option("ms_mon_service_mode", Option::TYPE_STR, Option::LEVEL_BASIC)
     .set_default("crc secure")
-    .set_description("Connection modes (crc, secure) for connections to mons in order of preference")
+    .set_description("Allowed connection modes (crc, secure) for connections to mons")
     .add_see_also("ms_service_mode")
     .add_see_also("ms_mon_cluster_mode")
     .add_see_also("ms_cluster_mode")
@@ -864,7 +864,7 @@ std::vector<Option> get_global_options() {
 
     Option("ms_service_mode", Option::TYPE_STR, Option::LEVEL_BASIC)
     .set_default("crc secure")
-    .set_description("Connection modes (crc, secure) for connections to daemons in order of preference")
+    .set_description("Allowed connection modes (crc, secure) for connections to daemons")
     .add_see_also("ms_cluster_mode")
     .add_see_also("ms_client_mode"),
 
diff --git a/src/msg/async/ProtocolV2.cc b/src/msg/async/ProtocolV2.cc
index 727e3aed426..234a0455d5f 100644
--- a/src/msg/async/ProtocolV2.cc
+++ b/src/msg/async/ProtocolV2.cc
@@ -2427,9 +2427,9 @@ CtPtr ProtocolV2::handle_auth_request(char *payload, uint32_t length) {
   std::vector<uint32_t> allowed_modes;
   messenger->auth_server->get_supported_con_modes(
     connection->get_peer_type(), auth_meta->auth_method, &allowed_modes);
-  for (auto mode : allowed_modes) {
-    if (std::find(preferred_modes.begin(), preferred_modes.end(), mode)
-	!= preferred_modes.end()) {
+  for (auto mode : preferred_modes) {
+    if (std::find(allowed_modes.begin(), allowed_modes.end(), mode)
+	!= allowed_modes.end()) {
       auth_meta->con_mode = mode;
       break;
     }
-- 
2.39.5