Bug 391528 - appdata screenshots broken
Summary: appdata screenshots broken
Status: RESOLVED FIXED
Alias: None
Product: Heaptrack
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Milian Wolff
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-07 16:07 UTC by Harald Sitter
Modified: 2018-05-04 16:46 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Harald Sitter 2018-03-07 16:07:43 UTC
appdata lists the following as screenshot

```
      <image>https://phabricator.kde.org/file/data/did3ip2xf6nut4srxany/PHID-FILE-lst6maof7cehaosav3e7/gui_summary.png</image>

```

which is coming back 404

```
curl -o /dev/null -D - -sSL https://phabricator.kde.org/file/data/did3ip2xf6nut4srxany/PHID-FILE-lst6maof7cehaosav3e7/gui_summary.png
HTTP/1.1 404 Not Found
Date: Wed, 07 Mar 2018 16:06:23 GMT
Server: Apache/2.4.18 (Ubuntu)
Set-Cookie: phsid=A%2Fsaq5i4vwhcfeammh4rt7goqilyjslidjfnwdnse4; expires=Mon, 06-Mar-2023 16:06:23 GMT; Max-Age=157680000; path=/; domain=phabricator.kde.org; secure; HttpOnly
X-Frame-Options: Deny
Strict-Transport-Security: max-age=0; includeSubdomains; preload
Content-Security-Policy: default-src 'self' https://phabricator.kde.org; img-src 'self' https://phabricator.kde.org data:; style-src 'self' https://phabricator.kde.org 'unsafe-inline'; script-src 'self' https://phabricator.kde.org; connect-src 'self'; frame-src 'self'; frame-ancestors 'none'; object-src 'none'; form-action 'self'; base-uri 'none'
Referrer-Policy: no-referrer
Cache-Control: no-store
Expires: Sat, 01 Jan 2000 00:00:00 GMT
X-Content-Type-Options: nosniff
Strict-Transport-Security: max-age=15768000
Transfer-Encoding: chunked
Content-Type: text/html; charset=UTF-8
```

Please fix and ideally switch all screenshots to the KDE product screenshot hosting @ https://phabricator.kde.org/source/websites-product-screenshots/browse/master/README.md putting strain on phabricator to serve static content is very meh.
Comment 1 Milian Wolff 2018-05-04 16:39:56 UTC
Git commit d4e2ad9f56b94b5f5089ef9941845614b3e0a18a by Milian Wolff.
Committed on 04/05/2018 at 16:39.
Pushed by mwolff into branch 'master'.

Add heaptrack screenshots

A  +-    --    heaptrack/gui_allocations_chart.png
A  +-    --    heaptrack/gui_flamegraph.png
A  +-    --    heaptrack/gui_summary.png

https://commits.kde.org/websites/product-screenshots/d4e2ad9f56b94b5f5089ef9941845614b3e0a18a
Comment 2 Milian Wolff 2018-05-04 16:46:11 UTC
Git commit a44b314095d510bb999f2c1e0200b50dbc64e1a7 by Milian Wolff.
Committed on 04/05/2018 at 16:41.
Pushed by mwolff into branch '1.1'.

Fix appstream screenshot URLs

Use the KDE CDN instead of phabricator.

M  +3    -3    src/analyze/gui/org.kde.heaptrack.appdata.xml

https://commits.kde.org/heaptrack/a44b314095d510bb999f2c1e0200b50dbc64e1a7