apt/test/integration/test-bug-602412-dequote-redirect, branch mainDebians commandline package managerhttps://git.kalnischkies.de/apt/atom?h=main2017-06-26T21:31:15Zdeal with 3xx httpcodes as required by HTTP/1.1 spec2017-06-26T21:31:15ZDavid Kalnischkiesdavid@kalnischkies.de2017-05-29T11:28:01Zurn:sha1:42654d08c2ca1bee18b6947a39228a35c2409deb
An unknown code should be handled the same as the x00 code of this
group, but for redirections we used to treat 300 (and a few others)
as an error while unknown codes were considered redirections.
Instead we check now explicitly for the redirection codes we support for
redirecting (and add the 308 defined in RFC 7538) to avoid future
problems if new 3xx codes are added expecting certain behaviours.
Potentially strange would have been e.g. "305 Use Proxy" sending a
Location for the proxy to use – which wouldn't have worked and resulted
in an error anyhow, but probably confused users in the process.
optional write aptwebserver log to client specific files2016-11-24T23:15:13ZDavid Kalnischkiesdavid@kalnischkies.de2016-11-24T11:14:39Zurn:sha1:e1ae0531bfad0fce8590c26d1e38825df22d812a
The test test-handle-redirect-as-used-mirror-change serves multiple
clients at the same time, so the order of the output is undefined and
once in a while the two clients will intermix their lines causing the
grep we perform on it later to fail making our tests fail.
Solved by introducing client-specific logfiles which we all grep and
sort the result to have the results more stable.
Git-Dch: Ignore
show final solution in --no-download --fix-missing mode2016-05-16T14:17:54ZDavid Kalnischkiesdavid@kalnischkies.de2016-05-14T08:43:03Zurn:sha1:eb1f04dda07c2b69549ad9fd793cca0e91841b3e
This commit moves the creation of the fetcher and with it the
calculation of the filenames before the code generation the various
lists detailing the solution. This means that simulation comes even so
slightly closer to a real run as it will require and parse the package
indexes for filenames and queuing of URIs, so that a simulation "using"
an unavailable download method actually fails now.
The real benefit of this change is through that the rather special but
nontheless handy --no-download --fix-missing mode now actually shows
what the solution is it will apply to the system rather than the
solution it would if it could download all not-downloaded packages.
test all redirection codes work as expected2016-01-31T18:06:19ZDavid Kalnischkiesdavid@kalnischkies.de2016-01-28T23:52:48Zurn:sha1:aa3e7b3287a20b464237c869483111abc7e9d815
Git-Dch: Ignore
tests: support spaces in path and TMPDIR2015-12-19T22:04:34ZDavid Kalnischkiesdavid@kalnischkies.de2015-12-15T16:20:26Zurn:sha1:3abb6a6a1e485b3bc899b64b0a1b7dc2db25a9c2
This doesn't allow all tests to run cleanly, but it at least allows to
write tests which could run successfully in such environments.
Git-Dch: Ignore
tests: don't use hardcoded port for http and https2015-09-15T08:16:09ZDavid Kalnischkiesdavid@kalnischkies.de2015-09-14T22:33:12Zurn:sha1:6c0765c096ffb4df14169236c865bbb2b10974ae
This allows running tests in parallel.
Git-Dch: Ignore
show item ID in Hit, Ign and Err lines as well2015-06-15T21:35:55ZDavid Kalnischkiesdavid@kalnischkies.de2015-06-15T14:41:43Zurn:sha1:1eb1836f4b5397497bd34f0cf516e6e4e73117bf
Again, consistency is the main sellingpoint here, but this way it is now
also easier to explain that some files move through different stages and
lines are printed for them hence multiple times: That is a bit hard to
believe if the number is changing all the time, but now that it keeps
consistent.
a hit on Release files means the indexes will be hits too2015-04-18T23:13:10ZDavid Kalnischkiesdavid@kalnischkies.de2015-04-12T15:08:46Zurn:sha1:ba6b79bd0090077724fa1272ea4d3a31706fcd5a
If we get a IMSHit for the Transaction-Manager (= the InRelease file or
as its still supported fallback Release + Release.gpg combo) we can
assume that every file we would queue based on this manager, but already
have locally is current and hence would get an IMSHit, too. We therefore
save us and the server the trouble and skip the queuing in this case.
Beside speeding up repetative executions of 'apt-get update' this way we
also avoid hitting hashsum errors if the indexes are in fact already
updated, but the Release file isn't yet as it is the case on well
behaving mirrors as Release files is updated last.
The implementation is a bit harder than the theory makes it sound as we
still have to keep reverifying the Release files (e.g. to detect now expired
once to avoid an attacker being able to silently stale us) and have to
handle cases in which the Release file hits, but some indexes aren't
present (e.g. user added a new foreign architecture).
improve https method queue progress reporting2015-04-18T23:13:08ZDavid Kalnischkiesdavid@kalnischkies.de2015-03-27T14:53:43Zurn:sha1:27925d82dd0cbae74d48040363fe6f6c2bae5215
The worker expects that the methods tell him when they start or finish
downloading a file. Various information pieces are passed along in this
report including the (expected) filesize. https was using a "global"
struct for reporting which made it 'reuse' incorrect values in some
cases like a non-existent InRelease fallbacking to Release{,.gpg}
resulting in a size-mismatch warning. Reducing the scope and redesigning
the setting of the values we can fix this and related issues.
Closes: 777565, 781509
Thanks: Robert Edmonds and Anders Kaseorg for initial patchs
test exitcode as well as string equality2015-03-16T17:01:54ZDavid Kalnischkiesdavid@kalnischkies.de2015-03-09T23:59:44Zurn:sha1:25b86db159fbc3c043628e285c0c1ef24dec2c6e
We use test{success,failure} now all over the place in the framework, so
its only consequencial to do this in the situations in which we test for
a specific output as well.
Git-Dch: Ignore