D22190: auto log in if username and password have been loaded from kwallet

Harald Sitter noreply at phabricator.kde.org
Mon Jul 1 15:45:57 BST 2019


sitter created this revision.
sitter added reviewers: Plasma, ngraham.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
sitter requested review of this revision.

REVISION SUMMARY
  it's a bit moot to load the data from the wallet and then still have the
  user manually click a log in button when we could just as well have tried
  to log in using the stored credentials.
  
  the new behavior is that the user goes through the bug reporting dialog,
  and when showing the login page it will automatically try to login using
  stored credentials. if the creds check out, the dialog automatically moves
  on to the next page.
  
  - page-validity is now in a helper canLogin function as it needs checking in two functions now (walletlogin and loginclick)
  - the page now automatically goes into login state and attempts to log in using stored credentials
  
  in the event that the credentials are wrong (e.g. password changed on
  website) the page will enter error state saying the credentials are wrong.
  when no credentials are in the wallet or the credentials are incomplete
  (password remembering is off) no auto login happens.

TEST PLAN
  - have no credentials -> no auto login
  - login manually and store credentials -> restart -> auto login happens
  - change password on website -> restart -> auto login fails & error is shown

REPOSITORY
  R871 DrKonqi

BRANCH
  auto-login

REVISION DETAIL
  https://phabricator.kde.org/D22190

AFFECTED FILES
  src/bugzillaintegration/reportassistantpages_bugzilla.cpp
  src/bugzillaintegration/reportassistantpages_bugzilla.h

To: sitter, #plasma, ngraham
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190701/0f8d9e1d/attachment-0001.html>


More information about the Plasma-devel mailing list