Merge pull request #137 from Rubc/add-connectivity-checker

feat: add connectivityChecker option
thephpleague · Aug 21, 2023 · c056bef · c056bef
2 parents 81be9e4 + 67e1697
commit c056bef
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/Adapter/Builder/SftpAdapterDefinitionBuilder.php b/src/Adapter/Builder/SftpAdapterDefinitionBuilder.php
@@ -85,6 +85,9 @@ protected function configureOptions(OptionsResolver $resolver)
 
  $resolver->setDefault('permPublic', 0744);
  $resolver->setAllowedTypes('permPublic', 'scalar');
+
+ $resolver->setDefault('connectivityChecker', null);
+ $resolver->setAllowedTypes('connectivityChecker', ['string', 'null']);
  }
 
  protected function configureDefinition(Definition $definition, array $options)
@@ -97,6 +100,10 @@ protected function configureDefinition(Definition $definition, array $options)
  $connectionFqcn = SftpConnectionProviderLegacy::class;
  }
 
+ if (null !== $options['connectivityChecker']) {
+ $options['connectivityChecker'] = new Definition($options['connectivityChecker']);
+ }
+
  $definition->setClass($adapterFqcn);
  $definition->setArgument(0,
  (new Definition($connectionFqcn))

diff --git a/tests/Adapter/Builder/SftpAdapterDefinitionBuilderTest.php b/tests/Adapter/Builder/SftpAdapterDefinitionBuilderTest.php
@@ -78,6 +78,7 @@ public function testOptionsBehavior()
  'directoryPerm' => 0755,
  'permPrivate' => 0700,
  'permPublic' => 0744,
+ 'connectivityChecker' => null,
  'host' => 'ftp.example.com',
  'username' => 'username',
  ];