Switch from download.k.o to autoconfig
Needs ReviewPublic

Authored by leinir on Jan 30 2020, 12:01 PM.

Details

Reviewers
bcooksley
Group Reviewers
Frameworks
KNewStuff
Summary

Rather than suggest using dko for provider files, we should be
suggesting autoconfig.k.o, which is a more capable service.

This is hopefully going to help in alleviating the overload on
the download.kde.org service, which does not handle the load as
well as autoconfig.kde.org does (and can't be fixed due to legacy).

See also https://marc.info/?l=kde-core-devel&m=158037601132219&w=2
for further details on the situation.

Diff Detail

Repository
R304 KNewStuff
Branch
fix-dko-overload-by-switching-to-autoconfig (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 22086
Build 22104: arc lint + arc unit
leinir created this revision.Jan 30 2020, 12:01 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 30 2020, 12:01 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
leinir requested review of this revision.Jan 30 2020, 12:01 PM
leinir updated this revision to Diff 74676.Jan 30 2020, 12:12 PM

As suggested by frinring, do not suggest setting a ProvidersUrl unless
it is actually needed.

  • Rework the knsrc description comment, suggest not setting ProvidersUrl
broulik added inline comments.
src/core/cache.cpp
163

We use https everywhere these days

leinir added inline comments.Jan 30 2020, 12:52 PM
src/core/cache.cpp
163

Yes, quite, we certainly do. (i'm also wondering if this is really something that wants changing or not, as it's... in a bit that loads KNS2 cache files... from KDE3)

leinir updated this revision to Diff 74687.Jan 30 2020, 1:04 PM

Thanks to broulik for noticing (or making me notice) this

  • Don't change a thing that deals with legacy data
leinir added inline comments.Jan 30 2020, 1:04 PM
src/core/cache.cpp
163

Yup, this wants to be left alone... i'll add a little comment to that effect.

meven added a subscriber: meven.Feb 3 2020, 7:58 AM
meven added inline comments.
tests/khotnewstuff.knsrc
2

Shouldn't it be in https ?

bcooksley added inline comments.Feb 3 2020, 7:59 AM
tests/khotnewstuff.knsrc
2

Yes, this should be https - the server will redirect to https (which sets HSTS headers) in any case.

meven added inline comments.Feb 3 2020, 8:04 AM
tests/khotnewstuff.knsrc
2

Better set this to https then, to make it explicit and not relying on server expectations.

leinir updated this revision to Diff 74896.Feb 3 2020, 9:17 AM

Address comments by meven and bcooksley

  • Make sure we're using https
leinir marked 6 inline comments as done.Feb 3 2020, 9:17 AM
leinir added inline comments.
tests/khotnewstuff.knsrc
2

Quite - that redirection isn't what these test knsrc files are supposed to be for, so good to get that fixed.

leinir updated this revision to Diff 75044.Feb 5 2020, 12:21 PM
leinir marked an inline comment as done.

During some cleanup and addition, i realised it was silly to have
the documentation for knsrc files in the widgets bits, and in two
places. In addition, those places were both at least a bit wrong,
or outdated, and needed fixing anyway, so... consolidate all that
into the README.md file, as it's fairly central anyway. More could
reasonably be done with that section, but this is already better.

  • Move the knsrc documentation to the central doc (as it's vital)
  • Minor doc fix for engine (which suggested a knsrc version 2 file)