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
[v12] SFTP fixes #24831
[v12] SFTP fixes #24831
Conversation
This change also greatly reduces the number of SFTP audit logs. Now SFTP events are only sent when files are opened or modified in any way, instead of for *every* SFTP request. * added to SFTP integration test * fix error when handling setstat on dirs * fix linter warning * move file/dir permission constants to lib/defaults package
Also fix copying symlinked files.
The sftp package is where modern file transfer logic lives and is being maintained. Make the web UI use this package to unify how we transfer files.
cd1aaf8
to
278c507
Compare
@capnspacehook Looks like this PR is missing |
I actually did tidy it, but I think when I did I forgot to update the |
Previously a Teleport client using SFTP would resolve remote host user home directories by making a subsystem request to a Teleport server which would return the home directory. The problem was the subsystem request counted as an open session, which could make the SFTP file transfer fail. This was frustrating and didn't make much sense, but after reading the SFTP specification again I realized that SFTP servers are to handle relative paths by assuming they start at the user's home directory. So let the server figure out the correct path and remove any tilde prefixes from remote paths.
278c507
to
0a37aba
Compare
Interesting, whenever I run |
Backports of: