Code review comment for ~ddstreet/maas:lp1916860-v2

Revision history for this message
Dan Streetman (ddstreet) wrote :

> While this fixes one of them it doesn't check for gaps in supported ciphers.

again, you're misunderstanding the api of ipmitool. the commit you landed to fix this is wrong.

> This branch will result push the settings string "XXXXXXXXXXXXXXX" which disables all ciphers making remote access impossible.

no, that isn't what this commit will use...

« Back to merge proposal