Re: [PATCH 00/15] Clean up credential asking code in SSH transport

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 1/17/23 10:20 AM, Peter Krempa wrote:
Peter Krempa (15):
   virNetLibsshSessionAuthAddPrivKeyAuth: Drop 'password' argument
   virNetLibsshAuthMethod: Drop 'password' field
   util: authconfig: Use automatic pointer clearing for virAuthConfig
   util: authconfig: Use conteporary and consistent header style
   virNetSSHSessionAuthAddPrivKeyAuth: Remove unused 'password' argument
   virNetSSHSessionAuthAddPrivKeyAuth: Refactor cleanup
   virNetSSHAuthMethod: Remove unused 'password' field
   virnetsshsession: Pass in username via virNetSSHSessionNew rather than
     auth functions
   util: auth: Introduce virAuthAskCredential
   virNetLibsshAuthenticateKeyboardInteractive: Use virAuthAskCredential
   virNetLibsshAuthenticatePrivkeyCb: Use virAuthAskCredential
   util: virauth: Export virAuthGetCredential
   virNetLibsshCheckHostKey: Use virAuthAskCredential
   virNetLibsshAuthenticatePassword: Use virAuthAskPassword instead of
     virAuthGetPasswordPath
   virAuthGetPasswordPath: Use virAuthAskCredential for callback
     interaction

  src/libvirt_private.syms      |   3 +
  src/rpc/virnetlibsshsession.c | 211 ++++++++++------------------------
  src/rpc/virnetlibsshsession.h |   3 +-
  src/rpc/virnetsocket.c        |  19 +--
  src/rpc/virnetsshsession.c    | 162 ++++++++------------------
  src/rpc/virnetsshsession.h    |  13 +--
  src/util/virauth.c            | 107 +++++++++++------
  src/util/virauth.h            |  12 ++
  src/util/virauthconfig.c      |  52 ++++-----
  9 files changed, 223 insertions(+), 359 deletions(-)



See separate emails for a few comments on individual patches

Reviewed-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux