Hi,
please find attached two small patches related to the conversion of the public key in a certificate to a public ssh-key.
The first fixes an issue which should only happen in master. The second might be useful for already released versions as well, but I would wait until we get a report that the old scheme fails with some certificates.
bye, Sumit
On Fri, Jun 17, 2016 at 02:50:36PM +0200, Sumit Bose wrote:
Hi,
please find attached two small patches related to the conversion of the public key in a certificate to a public ssh-key.
The first fixes an issue which should only happen in master. The second might be useful for already released versions as well, but I would wait until we get a report that the old scheme fails with some certificates.
bye, Sumit
ACK to both, sorry for not finding the first issue when I reviewed the original patch.
I also only tested the happy path with the second patch, not the issue that is actually fixed.
On Thu, Jul 07, 2016 at 06:13:04PM +0200, Jakub Hrozek wrote:
On Fri, Jun 17, 2016 at 02:50:36PM +0200, Sumit Bose wrote:
Hi,
please find attached two small patches related to the conversion of the public key in a certificate to a public ssh-key.
The first fixes an issue which should only happen in master. The second might be useful for already released versions as well, but I would wait until we get a report that the old scheme fails with some certificates.
bye, Sumit
ACK to both, sorry for not finding the first issue when I reviewed the original patch.
I also only tested the happy path with the second patch, not the issue that is actually fixed.
CI: http://sssd-ci.duckdns.org/logs/job/47/48/summary.html
The failure on rawhide is unrelated.
* master: * ecd48ae244dbb6490989752fba99b58d84babfa6 * 8b2bd0587af6ed6bbd7eab7a332ec88de6b7c36c
sssd-devel@lists.fedorahosted.org