<div dir="ltr"><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Sun, Jan 5, 2025 at 12:11 PM Albert Astals Cid <<a href="mailto:aacid@kde.org">aacid@kde.org</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
knewstuff - 2nd week<br>
 * <a href="https://invent.kde.org/frameworks/knewstuff/-/pipelines/856702" rel="noreferrer" target="_blank">https://invent.kde.org/frameworks/knewstuff/-/pipelines/856702</a><br>
  * knewstuff-atticaprovidertest is failing<br></blockquote><div><br></div><div>Something changed server side? Although our code is also meh.</div><div><br></div><div><a href="https://invent.kde.org/frameworks/knewstuff/-/blob/8e0c4d4042ab7ef5941ffbb270433576b6876028/src/core/provider.h#L89">https://invent.kde.org/frameworks/knewstuff/-/blob/8e0c4d4042ab7ef5941ffbb270433576b6876028/src/core/provider.h#L89</a></div><div><a href="https://invent.kde.org/frameworks/knewstuff/-/blob/8e0c4d4042ab7ef5941ffbb270433576b6876028/src/attica/atticarequester.cpp#L203">https://invent.kde.org/frameworks/knewstuff/-/blob/8e0c4d4042ab7ef5941ffbb270433576b6876028/src/attica/atticarequester.cpp#L203</a></div><div><a href="https://invent.kde.org/frameworks/attica/-/blob/ce4b34e1906c272df905f8b7833c28178f230b52/src/provider.h#L608">https://invent.kde.org/frameworks/attica/-/blob/ce4b34e1906c272df905f8b7833c28178f230b52/src/provider.h#L608</a></div><div><br></div><div>We coerce an int that uses -1 to indicate something (it's not documented.  I am also not sure what it could mean -- give me any page? give me the last page [but why would that be the default value]? give me no page? give me all pages? not initialized [but then which page should the request get]?) into an unsigned 4294967295 which the server then rejects as <br></div><div><br></div><div><ocs><meta><status>failed</status><statuscode>400</statuscode><message>Page out of range</message></meta></ocs></div><div><br></div><div>I'm a bit unsure how to best deal with this. Our default value of -1 doesn't really make sense to me, maybe we should just make it 0 or at least std::max(0, page) to preserve it as uninitialized state? The entire situation also leaves us also with a broader problem of how to reconcile knewstuff having an int page and attica having an unsigned page :|<br></div><div><br></div><div>HS<br></div><div><br></div></div></div>