https://bugs.kde.org/show_bug.cgi?id=414826

Tuomas Nurmi <tuo...@norsumanageri.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|REPORTED                    |RESOLVED
      Latest Commit|                            |https://invent.kde.org/mult
                   |                            |imedia/amarok/-/commit/588d
                   |                            |8cef9d760b86586afebdd58e8ee
                   |                            |32c77c09d

--- Comment #1 from Tuomas Nurmi <tuo...@norsumanageri.org> ---
Git commit 588d8cef9d760b86586afebdd58e8ee32c77c09d by Tuomas Nurmi.
Committed on 11/07/2024 at 21:46.
Pushed by nurmi into branch 'master'.

Update last.fm login mechanism to use non-deprecated token method

The mobile session seems to be somewhat deprecated, and at least I'm not
totally
confident that it is actually possible to log in new sessions with it any more
in
Amarok.

Token-based method doesn't need password, so no new data is saved to kwallet in
any
case. Tries to use existing session key from config file and username from
config
file or kwallet. If username is saved to kwallet, tries to migrate it from
there.
If this succeeds, everything should be transparent to user. If not, an error
window
is shown and the account can be reconnected in plugin settings.

Thus also avoids any KWallet related bugs

M  +1    -0    ChangeLog
M  +20   -28   src/services/lastfm/LastFmConfigWidget.ui
M  +9    -71   src/services/lastfm/LastFmService.cpp
M  +0    -2    src/services/lastfm/LastFmService.h
M  +15   -140  src/services/lastfm/LastFmServiceConfig.cpp
M  +2    -12   src/services/lastfm/LastFmServiceConfig.h
M  +122  -41   src/services/lastfm/LastFmServiceSettings.cpp
M  +4    -1    src/services/lastfm/LastFmServiceSettings.h

https://invent.kde.org/multimedia/amarok/-/commit/588d8cef9d760b86586afebdd58e8ee32c77c09d

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to