| Age | Commit message (Collapse) | Author | Lines |
|
They take a plain tar, which needs no compressor on the machine.
|
|
|
|
The notes test had borrowed its repository, so it now builds one.
|
|
Its message came from a here-document with a dash, which stripped the
tab git knows a Conflicts path by. git then found no trailer block at
all, so the test passed with the guard it is about removed. The message
is now written with printf.
|
|
repos/rob took commits, refs, tags, a repack and a moved HEAD from test
after test, so what a test checked hung on the ones before it. A test
that changes a repository, or needs one in a shape of its own, now
builds it, and rob is only read. The tests that leaned on a config or a
scanned repository an earlier test left build their own too.
|
|
Each now writes a types file and config of its own rather than
rewriting the one the test before it left.
|
|
Grouping folders first is tree page behaviour rather than a security
fix, so it joins the tree tests on a repository of its own.
|
|
The commit named is the one Git's v2.56.0 tag peels to upstream.
|
|
Each repository is built tree for tree as before. A helper that lost
the caller's change would still fail t0107, where following no path
shows file-1, but t0108 passed without its added line, so it now looks
for that line.
|
|
The configs a run of t0101 writes are byte for byte what they were. The
scan configs name no repository and descrc adds one to a copy of the
shared config, so those keep their own form.
|
|
A build without Lua left the eight tests of that pass out of the run,
where the rest of the script counts its Lua tests as skipped.
|
|
The id was written as 40 hex digits, which failed the test in a run
with GIT_TEST_DEFAULT_HASH=sha256.
|
|
|
|
|
|
A value cgit does not know as a period turns the statistics page off,
so a typo such as "decade" read the same as leaving stats off on
purpose. The empty value stays quiet because a cached repolist writes
it for a repository that turned stats off.
|
|
No test gave a file name invalid UTF-8 or a control byte, which APFS
refuses on disk, so the tree is built through the index.
|
|
The test ran gzip without the GZIP prerequisite t0110 uses for the same
need, and it took a file-1 anywhere in the listing rather than under the
tag's own prefix.
|
|
An unquoted $cgitbin breaks in a checkout whose path holds a space,
and head -c and head -1 are not POSIX. The status of the disconnected
request is still read right after its pipeline, which assumes the
shell waits for the left side as dash, bash, ksh and busybox ash all
do.
|
|
A loose ref written by hand needs the files backend, so under
GIT_TEST_DEFAULT_REF_FORMAT=reftable the broken tree test failed, and
the die test after it with it, since it borrowed that clone. It now
makes its own. Both index tests that stat a path through refs/heads
also passed there with their checks gone, as reftable keeps
refs/heads as a file. The commit is rewritten through files rather
than a pipeline that hid cat-file failing.
|
|
The expected content was "more" only because an earlier test had
rewritten file-1, and the next test reused that file and the tag. The
tag.gpgsign override did nothing, since test-lib already keeps the
user's config out.
|
|
The expected count of 2 held only because earlier tests had made
a.git and b.git and one of them had dropped b.git's cgitrc.
|
|
The count was bounded only from above, and loosely, so a page listing
no directories passed, as would a limit of 16. t0104's long directory
chain test already pins the bound exactly.
|
|
The blob came out of a pipeline whose failure would have left it empty
and the test passing, and nothing looked at the row itself. head -c is
not POSIX either.
|
|
HEAD was put back inside the && chain after cgit ran, so a failure left
repos/rob on a missing branch for every later test. The switcher check
also read the page this test left, and now runs in the same test.
|
|
Unlike its siblings the test never counted status lines, so a second
page after the first went unnoticed. The headers are out by the time
the diff is collected, so the one answer stays a 200 holding the
error.
|
|
cache_process bypasses a ttl of 0 and fails quietly in many ways, so
with a dead cache the test passed whatever ttl the page got. A second
config that keeps the same page for ever now has to replay it, from a
cache of its own, since slots are keyed on the request alone.
|
|
Every page ends in </html>, the 404 for no repositories included, and
an index, an error page or a page cut short lacks most things, so the
path-less repository, the empty url, the page without an auth filter
and the refused id each now show the page that was meant to render.
|
|
The loop asserted only absences, so a 404 or any error but the die
text passed, and the final 404 is also what a name that resolves to
nothing gets.
|
|
The test opened the inner tag by id and never looked at the outer
tag's page or the link the title is about.
|
|
The log walks the history before it prints a row, so nothing ahead of
the gap ever renders and the title promised more than the page shows.
The test also took its gap from the shared repository, where HEAD~2
was a given commit only because of tests run before it.
|
|
The email filter was handed a few bytes, which usually sit in the pipe
before the filter has even exited, so a cgit that died of SIGPIPE
passed most runs. A source filter handed more than a pipe holds has to
see the write fail.
|
|
git's read_mailmap_blob inflates the whole blob, so a pusher could make
every page of a repository read as much as they liked. git reads a
note and the .gitmodules file whole as well, and capping those would
mean bounding the lookups where the log, commit and tree pages make
them, so the manual names them instead. Nothing tested that
mailmap.file and mailmap.blob are ignored either.
|
|
A scanned repository sets its description, owner and section without
trust, and no test gave them any markup.
|
|
t0104's binary fixture never turns blame on, so neither the blame
page's own refusal nor the missing link on the file's page had a test.
|
|
The defbranch check had a test while its twin for a crafted HEAD,
which the index reads the same way to date a repository, had none.
|
|
Nothing in tests/ read robots.txt, so dropping the trailing slashes
again went unnoticed.
|
|
Only a benign query had been redirected, so nothing showed that a
space, a control byte or a byte past ASCII in the query string is
encoded before it reaches the Location header.
|
|
The plain page's header was the only one fed a newline. A snapshot
asked for with h= keeps the name it was requested under, and a blob
fetched by id takes any path, so both put visitor bytes into
Content-Disposition.
|
|
The title claimed every page, but only the log, commit and atom pages
were asked for. The commit now changes a file, since the patch page
leaves out a commit with nothing to show, and its parent is read
before the printf, where a failed rev-parse would have gone unnoticed.
|
|
The allowlist was tested in two places, and the 404 alone did not show
the transport was live behind the auth filter, since a page that does
not exist answers the same. A loose object and HEAD now have to come
through, and the planted alternates file is removed even when the test
fails, where it used to leave rob borrowing foo's objects.
|
|
Both requests went through the shared page cache, and the clone one
only worked because clone pages are never stored, which is what the
cgitrc_copy comment warns of.
|
|
points-at-tree only ever matched its entry in the branch switcher, and
points-at-tag was never looked for. Every branch also shared one date,
so the age and name sorts gave the same rows, and nothing showed the
name setting had applied. The loose refs written by hand need the
files backend.
|
|
The check read the page the previous test had left, and it took any
hex where the commit goes and only two of the eight conversions.
|
|
Only a lowercase javascript url was tried, so a denylist of that one
spelling would have passed. The quote test also read the page the test
before it had left.
|
|
Only %zz%2 had been sent, so nothing showed that markup in the query,
the hidden inputs a form carries, the errors naming a refused value or
the Host header is escaped on its way back.
|
|
For a bare q=% only the status was checked, so a decoder that dropped
the sign passed.
|
|
No test gave a ref name, a commit or tag message, a note or an ident
any markup, and the one hostile file name never reached an attribute.
html_txt leaves quotes alone, so a name holding both a quote and a
bracket is what tells an attribute written as text.
|
|
Nothing showed that every leading slash goes, through url= or
PATH_INFO, and one left behind joins the virtual root into a
scheme-relative link to another host.
|
|
All three shipped server configs hand cgit the path as PATH_INFO,
already decoded, yet every hostile path in the tests went through url=
instead.
|
|
The name also sits in the breadcrumb and the title, so the test passed
with the source filter never run, and the missing marker proved
nothing. dump.sh echoes its arguments ahead of the body it upper-cases,
which shows the name arrived whole as one argument.
|