Skip to content
Snippets Groups Projects

Ssh fingerprint fix

Merged gitlab-qa-bot requested to merge github/fork/raphendyr/ssh_fingerprint_fix into master

Created by: raphendyr

We should check exit status instead of look sting n output.

Someone need to validate the tests as I wrote them without testing...

I tested manually the code, so it works.

This should be merged to 4-1-stable and 4-2-stable too, as I think this is security update.

If this patch is not used, users might get gitolite (or any other) system in blocked state.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Created by: raphendyr

    Hmm. Did I broke tests or is there a bug... Damn..

    If somebody has a pointers, they are welcome.

    ps. I really need to get that test machine running...

    By Administrator on 2013-02-15T11:15:09 (imported from GitLab project)

    By Administrator on 2013-02-15T11:15:09 (imported from GitLab)

  • Created by: axilleas

    The travis tests are broken for some time now. I think I saw someone mentioning that it's postgresql's fault.

    By Administrator on 2013-02-15T14:15:51 (imported from GitLab project)

    By Administrator on 2013-02-15T14:15:51 (imported from GitLab)

  • Created by: raphendyr

    OK. Now travis errors are not from ssh stuff, I think. So I think I got ssh tests fixed.

    By Administrator on 2013-02-18T08:31:41 (imported from GitLab project)

    By Administrator on 2013-02-18T08:31:41 (imported from GitLab)

  • Created by: dzaporozhets

    thank you

    By Administrator on 2013-02-18T17:56:32 (imported from GitLab project)

    By Administrator on 2013-02-18T17:56:32 (imported from GitLab)

Please register or sign in to reply
Loading