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

[v13] ensure moderated file transfers only perform allowed operations #39356

Merged
merged 3 commits into from
Mar 15, 2024

Conversation

capnspacehook
Copy link
Contributor

Backport of #39091 and #39343.

changelog: only allow necessary operations during moderated file transfers and limit in-flight file transfer requests to one per session

capnspacehook and others added 2 commits March 14, 2024 10:05
* ensure moderated file transfers only preform allowed operations

* fix sess test

* wip integration test

* integration test working

* tweak some error messages and test cases

* allow setstat for uploads

* address nits from code review

* expand paths when necessary when creating file transfer request

* improve UX of errors in web UI

* deny request when an invalid user tries to transfer files

* don't allow open requests, they aren't done by the webui

* address feedback

* use a buffered reader when reading the file request, address a few nits
@capnspacehook capnspacehook added backport server-access sftp Issues related to Teleport's SFTP implementation labels Mar 14, 2024
@github-actions github-actions bot requested review from avatus, Tener and zmb3 March 14, 2024 16:06
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from avatus March 14, 2024 16:23
@capnspacehook capnspacehook added this pull request to the merge queue Mar 14, 2024
Merged via the queue into branch/v13 with commit e00ff25 Mar 15, 2024
22 checks passed
@capnspacehook capnspacehook deleted the capnspacehook/backport/v13/39091 branch March 15, 2024 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport server-access sftp Issues related to Teleport's SFTP implementation size/md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants