[SRU] plasma-distro-release-notifier gets EOL data from git.launchpad.net
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
plasma-distro-release-notifier (Ubuntu) |
Fix Released
|
Critical
|
Simon Quigley | ||
Jammy |
Fix Released
|
Critical
|
Simon Quigley | ||
Lunar |
Won't Fix
|
Undecided
|
Unassigned | ||
Mantic |
Won't Fix
|
Undecided
|
Unassigned | ||
Noble |
Fix Released
|
Critical
|
Simon Quigley | ||
Oracular |
Fix Released
|
Critical
|
Simon Quigley |
Bug Description
[Impact]
This isn't really an impact to users, but is certainly an impact to Launchpad's servers. plasma-
However, in its present form, plasma-
Simon Quigley has been kind enough to patch plasma-
Simon's work upstream has stalled-out. Instead I have moved the eol.json file to be a local file and pointed the URL to be a local file:// to get the file from the machine locally. Ideally it would just get the information from distro-info-data per Simon's patch, but this works for now.
[Test Case]
* Simply install Ubuntu Studio or Kubuntu, any supported version.
* Upon login, plasma-
* Simon's patch removes this requirement and uses on-disk distro-info-data instead, and my patch just gets the eol.json file locally.
* To trigger with the new version, Jammy will already be popping-up that 24.04 is available. For Noble, edit the on-disk json file at /usr/share/
[What could go wrong]
This notifier was originally used in KDE Neon, and as such, had to be patched to work with Ubuntu. KDE Neon still would pull from KDE's servers to get the json file to calculate the EOL dates This patch does rewrite a lot of that code, but seems to be solid nonetheless.
Attached is a screenshot of 22.04 in a VM, set to normal upgrades, and being "tricked" into thinking it's about to go EOL via date rewrites in /usr/share/
With the local eol.json file, this uses the QUrl class of Qt to grab the file:// URL of a local file. This has the same effect of grabbing it from another source except it does it locally and therefore prevents the DoS issue we were having on Launchpad. This is documented at https:/
description: | updated |
Changed in plasma-distro-release-notifier (Ubuntu Lunar): | |
status: | New → In Progress |
Changed in plasma-distro-release-notifier (Ubuntu Jammy): | |
status: | New → In Progress |
importance: | Undecided → High |
Changed in plasma-distro-release-notifier (Ubuntu Lunar): | |
importance: | Undecided → High |
Changed in plasma-distro-release-notifier (Ubuntu Jammy): | |
assignee: | nobody → Erich Eickmeyer (eeickmeyer) |
Changed in plasma-distro-release-notifier (Ubuntu Lunar): | |
assignee: | nobody → Erich Eickmeyer (eeickmeyer) |
Changed in plasma-distro-release-notifier (Ubuntu Jammy): | |
assignee: | Erich Eickmeyer (eeickmeyer) → Simon Quigley (tsimonq2) |
Changed in plasma-distro-release-notifier (Ubuntu Lunar): | |
assignee: | Erich Eickmeyer (eeickmeyer) → Simon Quigley (tsimonq2) |
Changed in plasma-distro-release-notifier (Ubuntu Mantic): | |
assignee: | Erich Eickmeyer (eeickmeyer) → Simon Quigley (tsimonq2) |
Changed in plasma-distro-release-notifier (Ubuntu Noble): | |
assignee: | Erich Eickmeyer (eeickmeyer) → Simon Quigley (tsimonq2) |
description: | updated |
Changed in plasma-distro-release-notifier (Ubuntu Oracular): | |
status: | In Progress → Fix Committed |
Changed in plasma-distro-release-notifier (Ubuntu Oracular): | |
status: | Fix Released → In Progress |
Changed in plasma-distro-release-notifier (Ubuntu Lunar): | |
assignee: | Simon Quigley (tsimonq2) → nobody |
Changed in plasma-distro-release-notifier (Ubuntu Mantic): | |
assignee: | Simon Quigley (tsimonq2) → nobody |
Changed in plasma-distro-release-notifier (Ubuntu Lunar): | |
importance: | High → Undecided |
Changed in plasma-distro-release-notifier (Ubuntu Mantic): | |
importance: | High → Undecided |
description: | updated |
Simon's original patch. I will be doing the upload, so this is just here for informational purposes.