Installed Flysystem with SFTP adapter and when testing found that if `Cache filesystem metadata` (in settings below) is set to true, the file would not successfully upload and behaviour is a little b0rked. By that I mean the file_managed table would be populated, the file would be on the SFTP server but there would be no file attached to the entity field and no failure message to the user.
$schemes = [
'sftpexample' => [
'driver' => 'sftp',
'config' => [
'host' => 'example.com',
'username' => 'username',
'password' => 'password', // Only one of 'password' or 'privatekey' is needed.
'privateKey' => 'path/to/or/contents/of/privatekey',
'root' => '/path/to/root',
// Optional
'port' => 21,
'timeout' => 10,
],
'cache' => TRUE, // Cache filesystem metadata.
],
];
Tracing through I tracked it to:
DrupalCacheAdapter.php:
/**
* {@inheritdoc}
*/
public function setVisibility($path, $visibility) {
$metadata = $this->adapter->setVisibility($path, $visibility);
return $this->updateMetadata($path, $metadata);
}
The above should return a metadata array for the cacheItem but instead with the SFTP adapter at least it returns an integer. This is owing to the adapter for SFTP and the result of chmod from the League\flysystem-sftp package:
public function setVisibility($path, $visibility)
{
....
return $connection->chmod($this->{'perm'.$visibility}, $path);
}
Changing DrupalCacheAdapter.php to the below resolves:
/**
* {@inheritdoc}
*/
public function setVisibility($path, $visibility) {
$this->adapter->setVisibility($path, $visibility);
return $this->updateMetadata($path, $this->adapter->getVisibility($path));
}
I've attached a patch, although I haven't had the time to look at the various adapters and see if this an approach that could work for all.
BTW. Very impressed by FlySystem and the flexibility it offers!
Hopefully this helps someone.
Thanks
Active
2.1
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Evaluate for relevancy, check adapter interface for requirements.