ALV-175: Apply "Chroot" configuration to filetransfer operations #24
Conversation
The naming is confusing (it's not trying to follow chroot semantics it just disallow certain path prefixes), and it will be replaced by the "Chroot" option in root of the config object. Signed-off-by: Alf-Rune Siqveland <alf.rune@northern.tech>
|
Just to double check - the idea presented in the ALV-175 is that the nt-connect will automatically add CONNECT_CHROOT value as a prefix to all the downloads and uploads, right? |
|
also, if the file already exists, I'm getting |
The implementation is definitely broken. It's only evaluating the constraint without applying the actual chroot local path. |
|
@kjaskiewiczz I updated the implementation and refactored the path expansions to session/model/filetransfer.go. |
Changelog: Title Ticket: ALV-175 Signed-off-by: Alf-Rune Siqveland <alf.rune@northern.tech>
Signed-off-by: Alf-Rune Siqveland <alf.rune@northern.tech>
This causes the application to panic every time the connection is closed. Changelog: None Ticket: None Signed-off-by: Alf-Rune Siqveland <alf.rune@northern.tech>
kjaskiewiczz
left a comment
kjaskiewiczz
left a comment
There was a problem hiding this comment.
looks good;
I also tested it with my edge device (raspberry pi4) on staging and it was working as expected
2 participants
No description provided.