Skip to content

Commit 83196d9

Browse files
Allow null password
1 parent b00803d commit 83196d9

2 files changed

Lines changed: 3 additions & 4 deletions

File tree

src/Adapter/Builder/SftpAdapterDefinitionBuilder.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,8 @@ protected function configureOptions(OptionsResolver $resolver)
4343
$resolver->setRequired('username');
4444
$resolver->setAllowedTypes('username', 'string');
4545

46-
$resolver->setRequired('password');
47-
$resolver->setAllowedTypes('password', 'string');
46+
$resolver->setDefault('password', null);
47+
$resolver->setAllowedTypes('password', ['string', 'null']);
4848

4949
$resolver->setDefault('port', 22);
5050
$resolver->setAllowedTypes('port', 'scalar');

tests/Adapter/Builder/SftpAdapterDefinitionBuilderTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ public function provideValidOptions()
2727
yield 'minimal' => [[
2828
'host' => 'ftp.example.com',
2929
'username' => 'username',
30-
'password' => 'password',
3130
]];
3231

3332
yield 'full' => [[
@@ -65,6 +64,7 @@ public function testOptionsBehavior()
6564
]);
6665

6766
$expected = [
67+
'password' => 'password',
6868
'port' => 22,
6969
'root' => '/path/to/root',
7070
'privateKey' => '/path/to/or/contents/of/privatekey',
@@ -74,7 +74,6 @@ public function testOptionsBehavior()
7474
'permPublic' => 0744,
7575
'host' => 'ftp.example.com',
7676
'username' => 'username',
77-
'password' => 'password',
7877
];
7978

8079
$this->assertSame(SftpAdapter::class, $definition->getClass());

0 commit comments

Comments
 (0)