Problem/Motivation

I can't get SFTP destinations working in Drupal 9 for reasons:

  1. 'drupal/flysystem_sftp' is abandoned. It depends on Drupal 8.
  2. I've submitted a MR to move this code to 'drupal/flysystem': #3354667: Add SFTP support to core module. Makes sense to include it, there is already FTP support.
  3. SFTP support in 'league/flysystem' is now provided by a different package: 'league/flysystem-sftp-v3' which requires league/flysystem 3.
  4. I can't require that library because it breaks drupal/backup_migrate_flysystem because it requires 2.x

This means I can't add SFTP support at all right now on anything higher than Drupal 8!

Steps to reproduce

On a Drupal 9 codebase:

composer require drupal/backup_migrate_flysystem drupal/flysystem_sftp

Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - drupal/flysystem_sftp[dev-1.x, 1.x-dev] require drupal/core ~8.0 -> found drupal/core[8.0.0-beta6, ..., 8.9.x-dev] but the package is fixed to 9.5.3 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.

Proposed resolution

Add 3.x branch of flysystem as a possible dependency.

Remaining tasks

A release would be nice.

User interface changes

None

API changes

None.

There are no significant changes in the 3.x branch: https://git.drupalcode.org/project/flysystem/-/compare/2.0.x...3.0.x?fro...

Data model changes

None

Issue fork backup_migrate_flysystem-3358974

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Jon Pugh created an issue. See original summary.

jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented
Issue summary: View changes
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented

I got composer to work using an "alias":

composer require drupal/flysystem:"3.0.x-dev as 2.0.0-beta1"

Jon Pugh opened merge request !4

  • JonMcL committed c9232974 on 5.0.x authored by Jon Pugh 
    Issue #3358974: Can't use SFTP for Backup Migrate in Drupal 9 due to...
jonmcl’s picture

jonmcl commented
Version: 5.0.x-dev » 5.0.1
Category: Bug report » Task
Status: Active » Fixed
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented

Yay!!

Thanks JonMcL!

jonmcl’s picture

jonmcl commented

No problem, other Jon :)

I didn't do much testing, primarily because flysystem_s3 need updates to work with the flysystem 3.0.x branch and my primary use case is S3. Please open another issue if you find problems. I'd be happy to investigate.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.