Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new ftp backend #27194

Merged
merged 8 commits into from
Oct 14, 2021
Merged

new ftp backend #27194

merged 8 commits into from
Oct 14, 2021

Conversation

icewind1991
Copy link
Member

@icewind1991 icewind1991 commented May 27, 2021

New FTP storage backend that instead of using php's ftp streamwrapper directly uses the ftp_ methods.
This removes some of the restrictions of the php stream wrapper (most notably being the inability to work with anything with a # in the filename) and gives more flexibility for working around some incompatibilities between server implementations.

Additionally this adds a new testing setup to test the ftp implementation against various ftp servers to ensure that all the various implementation differences are accounted for. The tests are setup to run only when files_external is touched to prevent wasting github's cpu cycles.

@icewind1991 icewind1991 added the 2. developing Work in progress label May 27, 2021
@icewind1991 icewind1991 force-pushed the new-ftp branch 12 times, most recently from 9389433 to a3f893e Compare May 31, 2021 13:27
@icewind1991 icewind1991 changed the title [wip] new ftp backend new ftp backend May 31, 2021
@icewind1991 icewind1991 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels May 31, 2021
@MorrisJobke MorrisJobke removed their request for review July 4, 2021 11:34
this uses the raw `ftp_` functions instead of the stream wrapper

Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
.github/workflows/ftp.yml Outdated Show resolved Hide resolved
Signed-off-by: Robin Appelman <[email protected]>
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 looks good to me in general, assuming the tests cover the raw + mlsd cases

would be good to check the code style checks

Signed-off-by: Robin Appelman <[email protected]>
@skjnldsv skjnldsv added this to the Nextcloud 23 milestone Oct 8, 2021
@ChristophWurst ChristophWurst removed their request for review October 11, 2021 08:06
@skjnldsv skjnldsv mentioned this pull request Oct 13, 2021
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 14, 2021
@skjnldsv skjnldsv merged commit 739b670 into master Oct 14, 2021
@skjnldsv skjnldsv deleted the new-ftp branch October 14, 2021 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement feature: external storage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants