Skip to content

Commit 6472b9b

Browse files
committed
mgr/smb: Use the default for minimum protocol version
'server min protocol' smb.conf option indicates the minimum SMB protocol version that the server can support during client negotiation phase. We can be generous in this case to accept various client implementations with a subset of supported protocol versions. With 'SMB2' it selects the highest available version from the list of sub protocols. Instead we refrain from configuring the option to support the least possible SMB2 version by default(which is SMB2_02 at this point in time) from Samba. ref: https://www.samba.org/samba/docs/current/man-html/smb.conf.5.html Signed-off-by: Anoop C S <[email protected]>
1 parent 5ac43ef commit 6472b9b

File tree

4 files changed

+0
-4
lines changed

4 files changed

+0
-4
lines changed

qa/suites/orch/cephadm/smb/tasks/deploy_smb_basic.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ tasks:
5252
"read only" = "no"
5353
"browseable" = "yes"
5454
[globals.default.options]
55-
"server min protocol" = "SMB2"
5655
"load printers" = "no"
5756
"printing" = "bsd"
5857
"printcap name" = "/dev/null"

qa/suites/orch/cephadm/smb/tasks/deploy_smb_domain.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ tasks:
5151
"read only" = "no"
5252
"browseable" = "yes"
5353
[globals.default.options]
54-
"server min protocol" = "SMB2"
5554
"load printers" = "no"
5655
"printing" = "bsd"
5756
"printcap name" = "/dev/null"

src/pybind/mgr/smb/handler.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1069,7 +1069,6 @@ def _generate_config(
10691069
'globals': {
10701070
'default': {
10711071
'options': {
1072-
'server min protocol': 'SMB2',
10731072
'load printers': 'No',
10741073
'printing': 'bsd',
10751074
'printcap name': '/dev/null',

src/pybind/mgr/smb/tests/test_smb.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -446,7 +446,6 @@ def test_share_dump_config(tmodule):
446446
'globals': {
447447
'default': {
448448
'options': {
449-
'server min protocol': 'SMB2',
450449
'load printers': 'No',
451450
'printing': 'bsd',
452451
'printcap name': '/dev/null',

0 commit comments

Comments
 (0)