Merge lp:~jakobunt/ltsp/ldm-log-v2 into lp:~ltsp-upstream/ltsp/ldm-trunk
Proposed by
Jakob Unterwurzacher
Status: | Merged |
---|---|
Merge reported by: | Vagrant Cascadian |
Merged at revision: | not available |
Proposed branch: | lp:~jakobunt/ltsp/ldm-log-v2 |
Merge into: | lp:~ltsp-upstream/ltsp/ldm-trunk |
Diff against target: |
130 lines (+40/-18) 1 file modified
src/plugins/ssh/ssh.c (+40/-18) |
To merge this branch: | bzr merge lp:~jakobunt/ltsp/ldm-log-v2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Vagrant Cascadian | Pending | ||
Review via email: mp+235541@code.launchpad.net |
Commit message
Bye bye "No response from server" - add ssh error logging
ldm now reads the full output from ssh to show a proper error message
to the user (and also write it to the log).
Description of the change
v2 of the "Bye bye No response from server" patch splits up the change into eight small chunks.
It also tries to be as little intrusive as possible.
Each patch has been individually tested (compile, successful login, incorrect password).
Thanks,
Jakob
To post a comment you must log in.
Thanks for the reworked patches!
Having tested this patchset, it does work to resolve the incorrect loging or password issue with feedback. It does seem to introduce a little noise on a successful login that flashes by too fast to read...
Overall, I think this is better than what we've had before.