Spdk/lib
Shuhei Matsumoto 0bb86d1cf3 iscsi: Fix the bug that mutual chap is not enabled by .ini config file
As long as source code and sample config file are checked, mutual
chap for discovery sessions should be enabled by either "Mutual"
or "CHAP Mutual". Especially SPDK iSCSI guides users to use not
"Mutual" but "CHAP Mutual".

The reason is that .INI config dump outputs "DiscoveryAuthMethod CHAP Mutual"
when g_spdk_iscsi.mutual_chap is true.

However "DiscoveryAuthMethod CHAP Mutual" doesn't work as expected.
When it is specified, g_spdk_iscsi.require_chap becomes true but
g_spdk_iscsi.mutual_chap becomes false.

When either "DiscoveryAuthMethod Mutual" or "DiscoveryAuthMethod Mutual CHAP"
is specified, both g_spdk_iscsi.require_chap and g_spdk_iscsi.mutual_chap
become true as expected. But any user cannot guess "Mutual CHAP".

On the other hand, CHAP feature for iSCSI target nodes works as
expected.

Fix this issue according to the implementation in iSCSI target nodes.

One difference between iSCSI target node and discovery service remains
that iSCSI target node doesn't allow "AuthMethod Mutual" but discovery
service allows "DiscoveryAuthMethod Mutual" to be compatible.

Change-Id: Ia3ce1640ffb1303ea77a05009bcabddaff080941
Signed-off-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-on: https://review.gerrithub.io/423963
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-09-11 21:59:44 +00:00
..
bdev bdev/raid: add reset handling 2018-09-11 19:00:29 +00:00
blob blobstore: Add support for bdev_blob_queue_io in blobstore 2018-09-11 21:19:34 +00:00
blobfs thread: Add a name parameter to spdk_register_io_device 2018-09-05 16:00:54 +00:00
conf conf: don't strdup name if section already exist 2018-05-17 17:13:19 +00:00
copy thread: Add a name parameter to spdk_register_io_device 2018-09-05 16:00:54 +00:00
env_dpdk env/dpdk: add security checks to spdk_env_get_socket_id() 2018-09-10 17:23:39 +00:00
event framework: fix null deref if no tty 2018-09-10 17:28:14 +00:00
ioat ioat: change return type of spdk_ioat_detach() to be void 2018-09-11 21:16:48 +00:00
iscsi iscsi: Fix the bug that mutual chap is not enabled by .ini config file 2018-09-11 21:59:44 +00:00
json json: Add spdk_json_decode_uint16 2018-06-05 21:30:02 +00:00
jsonrpc jsonrpc: fix error path of spdk_jsonrpc_parse_request() 2018-08-31 17:41:58 +00:00
log app: allow full-name command line options 2018-08-13 17:30:10 +00:00
lvol lvol: move null check from verify_lvol_name 2018-09-05 15:59:45 +00:00
nbd nbd: Encapsulate spdk_event usage in subsystem module 2018-09-05 19:32:37 +00:00
net lib: fix typos in the lib directory 2018-08-24 17:15:12 +00:00
nvme nvme: add ERROR controller states in initialization 2018-09-07 20:03:29 +00:00
nvmf nvmf: get qp_context only on QP related event 2018-09-11 21:33:39 +00:00
rocksdb blobfs: add a new API to return file's unique ID 2018-08-29 16:29:22 +00:00
rpc rpc: g_rpc_lock_path: remove redundant plus 2018-09-04 22:21:04 +00:00
scsi trace: remove alias concept 2018-09-05 18:03:43 +00:00
sock sock: fix socket memory free 2018-08-10 15:38:38 +00:00
thread thread: Fix issue where a channel would be destroyed twice 2018-09-05 16:00:54 +00:00
trace trace: remove alias concept 2018-09-05 18:03:43 +00:00
ut_mock test: Simplify the mock library 2018-08-09 17:18:23 +00:00
util Clean up some incorrect Copyright statements 2018-07-30 16:29:38 +00:00
vhost vhost: Stop memory leak in external foreach loop 2018-08-31 19:09:07 +00:00
virtio virtio: clarify feature negotiation failure 2018-08-01 19:16:54 +00:00
Makefile test: remove spdk_cunit library 2018-07-06 18:35:03 +00:00