push @$cmd, 'guest,username=guest';
}
- push @$cmd, '-o', defined($smbver) ? "vers=$smbver" : "vers=3.0";
+ push @$cmd, '-o', defined($smbver) ? "vers=$smbver" : "vers=default";
run_command($cmd, errmsg => "mount error");
}
maxLength => 256,
},
smbversion => {
- description => "SMB protocol version",
+ description => "SMB protocol version. 'default' if not set, negotiates the highest SMB2+"
+ ." version supported by both the client and server.",
type => 'string',
- enum => ['2.0', '2.1', '3.0'],
+ default => 'default',
+ enum => ['default', '2.0', '2.1', '3', '3.0', '3.11'],
optional => 1,
},
};
disable => { optional => 1 },
maxfiles => { optional => 1 },
'prune-backups' => { optional => 1 },
+ 'max-protected-backups' => { optional => 1 },
content => { optional => 1 },
format => { optional => 1 },
username => { optional => 1 },
smbversion => { optional => 1},
mkdir => { optional => 1 },
bwlimit => { optional => 1 },
+ preallocation => { optional => 1 },
};
}
# Storage implementation
sub on_add_hook {
- my ($class, $storeid, $scfg, %param) = @_;
+ my ($class, $storeid, $scfg, %sensitive) = @_;
- if (defined($param{password})) {
- cifs_set_credentials($param{password}, $storeid);
+ if (defined($sensitive{password})) {
+ cifs_set_credentials($sensitive{password}, $storeid);
if (!exists($scfg->{username})) {
- warn "ignoring password parameter\n";
+ warn "storage $storeid: ignoring password parameter, no user set\n";
}
} else {
cifs_delete_credentials($storeid);
}
sub on_update_hook {
- my ($class, $storeid, $scfg, %param) = @_;
+ my ($class, $storeid, $scfg, %sensitive) = @_;
- return if !exists($param{password});
+ return if !exists($sensitive{password});
- if (defined($param{password})) {
- cifs_set_credentials($param{password}, $storeid);
+ if (defined($sensitive{password})) {
+ cifs_set_credentials($sensitive{password}, $storeid);
if (!exists($scfg->{username})) {
- warn "ignoring password parameter\n";
+ warn "storage $storeid: ignoring password parameter, no user set\n";
}
} else {
cifs_delete_credentials($storeid);
my $servicename = '//'.$scfg->{server}.'/'.$scfg->{share};
- my $cmd = ['/usr/bin/smbclient', $servicename, '-d', '0', '-m'];
+ my $cmd = ['/usr/bin/smbclient', $servicename, '-d', '0'];
- push @$cmd, $scfg->{smbversion} ? "smb".int($scfg->{smbversion}) : 'smb3';
+ if (defined($scfg->{smbversion}) && $scfg->{smbversion} ne 'default') {
+ # max-protocol version, so basically only relevant for smb2 vs smb3
+ push @$cmd, '-m', "smb" . int($scfg->{smbversion});
+ }
if (my $cred_file = get_cred_file($storeid)) {
push @$cmd, '-U', $scfg->{username}, '-A', $cred_file;
if (my $err = $@) {
die "$out_str\n" if defined($out_str) &&
- ($out_str =~ m/NT_STATUS_ACCESS_DENIED/);
+ ($out_str =~ m/NT_STATUS_(ACCESS_DENIED|LOGON_FAILURE)/);
return 0;
}
return 1;
}
+# FIXME remove on the next APIAGE reset.
+# Deprecated, use get_volume_attribute instead.
sub get_volume_notes {
my $class = shift;
PVE::Storage::DirPlugin::get_volume_notes($class, @_);
}
+
+# FIXME remove on the next APIAGE reset.
+# Deprecated, use update_volume_attribute instead.
sub update_volume_notes {
my $class = shift;
PVE::Storage::DirPlugin::update_volume_notes($class, @_);
}
+sub get_volume_attribute {
+ return PVE::Storage::DirPlugin::get_volume_attribute(@_);
+}
+
+sub update_volume_attribute {
+ return PVE::Storage::DirPlugin::update_volume_attribute(@_);
+}
+
1;