{"action":"created","issue":{"url":"https://api.github.com/repos/bit-team/backintime/issues/532","repository_url":"https://api.github.com/repos/bit-team/backintime","labels_url":"https://api.github.com/repos/bit-team/backintime/issues/532/labels{/name}","comments_url":"https://api.github.com/repos/bit-team/backintime/issues/532/comments","events_url":"https://api.github.com/repos/bit-team/backintime/issues/532/events","html_url":"https://github.com/bit-team/backintime/pull/532","id":129329555,"number":532,"title":"Fuse mounts for SSH, Local Encrypted, and SSH encrypted are read-only by default","user":{"login":"dinoboy197","id":631178,"avatar_url":"https://avatars.githubusercontent.com/u/631178?v=3","gravatar_id":"","url":"https://api.github.com/users/dinoboy197","html_url":"https://github.com/dinoboy197","followers_url":"https://api.github.com/users/dinoboy197/followers","following_url":"https://api.github.com/users/dinoboy197/following{/other_user}","gists_url":"https://api.github.com/users/dinoboy197/gists{/gist_id}","starred_url":"https://api.github.com/users/dinoboy197/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dinoboy197/subscriptions","organizations_url":"https://api.github.com/users/dinoboy197/orgs","repos_url":"https://api.github.com/users/dinoboy197/repos","events_url":"https://api.github.com/users/dinoboy197/events{/privacy}","received_events_url":"https://api.github.com/users/dinoboy197/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2016-01-28T04:10:43Z","updated_at":"2016-02-04T18:26:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bit-team/backintime/pulls/532","html_url":"https://github.com/bit-team/backintime/pull/532","diff_url":"https://github.com/bit-team/backintime/pull/532.diff","patch_url":"https://github.com/bit-team/backintime/pull/532.patch"},"body":"@Germar , per our email discussion this is part of the precursor work to moving towards full-rsync mode for all backup modes.\r\n\r\nI thought it would be prudent to add read-only for mounts through fuse as a default constructor argument to the ```Mount``` object, in case there is a time in the future when mounting in non-read-only mode is desired. The UI or some other method calling into the ```Mount``` class could pass the read-only flag as they desire.\r\n\r\nOn testing - I'm not sure what the best way to unit-test or end-to-end test this is at the moment. I think I'd have to set up ssh for the test rig, which I think could be a bigger effort which might be better isolated in a different piece of work? What do you think?\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/bit-team/backintime/issues/comments/179981706","html_url":"https://github.com/bit-team/backintime/pull/532#issuecomment-179981706","issue_url":"https://api.github.com/repos/bit-team/backintime/issues/532","id":179981706,"user":{"login":"Germar","id":2731999,"avatar_url":"https://avatars.githubusercontent.com/u/2731999?v=3","gravatar_id":"","url":"https://api.github.com/users/Germar","html_url":"https://github.com/Germar","followers_url":"https://api.github.com/users/Germar/followers","following_url":"https://api.github.com/users/Germar/following{/other_user}","gists_url":"https://api.github.com/users/Germar/gists{/gist_id}","starred_url":"https://api.github.com/users/Germar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Germar/subscriptions","organizations_url":"https://api.github.com/users/Germar/orgs","repos_url":"https://api.github.com/users/Germar/repos","events_url":"https://api.github.com/users/Germar/events{/privacy}","received_events_url":"https://api.github.com/users/Germar/received_events","type":"User","site_admin":false},"created_at":"2016-02-04T18:26:42Z","updated_at":"2016-02-04T18:26:42Z","body":"Thanks for explanation.\r\n\r\n> So now, running that snapshot unit test for changing file group fails because running them with just plain python won't give python the ability to change the group - the operating system in travis blocks it.\r\n\r\nActually it's not the operating system blocking this but after adding a user to a new group the user need to log out and log in again to apply that new group. So this made me curious on why your new `TestSSH` test could work without log out and log in!? And I just found out that at least on Ubuntu 14.04 `fuse` membership is not necessary anymore to mount a FuseFS. Even more in Ubuntu 15.10 `fuse` group is gone completely.\r\n\r\nI will set up a new 12.04 VM later this evening to test how `fuse` membership was handled in there (as Travis is running 12.04). So if `fuse` membership isn't necessary anymore for mounting a FuseFS we could skip that test in BIT."}}