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

Fix FTP not properly parsing lines with sticky/SUID/SGID permissions #473

Merged
merged 3 commits into from
Mar 26, 2021

Conversation

althonos
Copy link
Member

@althonos althonos commented Mar 26, 2021

Type of changes

  • Bug fix

Checklist

  • I've run the latest black with default args on new code.
  • I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
  • I've added tests for new code.
  • I accept that @PyFilesystem/maintainers may be pedantic in the code review.

Description

The Linux regex in fs._ftp_parse was unable to recognize sticky/SUID/SGID bits in permissions. Closes #451, which was caused by a SGID bit. I also fixed the regex so that is can recognize more Unix file types, instead of just -, l or d as before.

@coveralls
Copy link

coveralls commented Mar 26, 2021

Coverage Status

Coverage remained the same at 95.362% when pulling 7f76454 on fix-451 into 4efe083 on master.

@althonos althonos added the bug label Mar 26, 2021
@althonos althonos added this to the v2.4.13 milestone Mar 26, 2021
@althonos althonos merged commit 60b0a63 into master Mar 26, 2021
@althonos althonos deleted the fix-451 branch March 26, 2021 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FTPFS does not list certain folders
2 participants