Commits
- Commit:
c2f5b3e63df878c687928a616ef24e87131a795e
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: avoid unnecessary strcmp in gotweb_assign_querystring()
- Commit:
1552435e8b0c7c7d4bcbb29087e912debeed3388
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: lower the level of the qs ptr-to-ptr
- Commit:
d5c153d0d827ef4993f41fca5a6eff8736e85a76
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: remove previd and prevset query string parameter
They're no longer used.
- Commit:
9ba23f168f80cb5a70c47b1a937de20515bfc118
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: delete dead code
at this point we can't know if the client has disconnected, we just
finished parsing the fcgi parameters and started handling the reply.
We'll know whether the client disconnected only later, while trying
to send the page content.
- Commit:
2a83fef7ce7879df1f318a442bb1d8d0012fa5e8
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: use less temp files
Instead of allocating a whole set of temp fd per `listen' directive,
just use a global set per "sockets" process. This is fine since
gotwebd doesn't keep multiple connections open at the same time,
once it enters gotwebd_process_request() it won't exit until all
the page has been processed. (this because we don't have async
APIs for got operations.)
ok stsp@
- Commit:
1220d7ea84811c79f71a7f24917d77cb2f10b02a
- From:
- Omar Polo <op@omarpolo.com>
- Date:
got{,web}d: unify log.c
gotd and gotwebd are using almost the same log.c file, so unify.
This only slightly changes gotwebd logs to include the process name,
like gotd.
ok stsp@
- Commit:
2959f41846dadc2d7826083bfc86d41a49b37420
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: add the magic ".git" handling
like gotd, assume an implicit ".git" if the repo doesn't exist.
Merge gotweb_init_repo_dir() and gotweb_load_got_path() since the
former only sets repo_dir->name, and we don't realistically know
the name until we've opened the directory.
While here, unbreak the git worktree handling too.
Realistically, moving forward, we should issue a redirect to the
canonical URL or at least set the correct metadata in the document
to avoid having multiple URLs for the same content.
ok stsp; tiny change to repo_dir->name handling done after the ok.
- Commit:
26b163a03e920e099a9aeefaeb6e99d0e0a8baf9
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: guard against missig folder and file parameter in BLOB and BLAME
ok stsp
- Commit:
e114f3d1c194932aef73612868f0bedc33110110
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: retire max_repos setting
Unlike max_repos_display which limits the number of items per page,
max_repos is an hard-limit on the number of repos gotwebd would
process.
"kill it with fire" tracey@ about the idea
ok stsp
- Commit:
8762929a1a8555f4108c4495bd52ca6dd7f7aefc
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: add knob for the number of tags and commits in the summary page
ok jamsek
- Commit:
5d6193d0d085d8d8ab3c2f6357b3121bbe728a50
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: remove PAGE handling
The page querystring parameter is long gone and currently unused.
It was replaced mith the "more" buttons in all the views, except
INDEX that uses `index_page'.
ok jamsek
- Commit:
0c6bdfca5ba9050d77d0984aee14f03aabd64bfc
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: use nitems() instead of ACTIONS__MAX
I feel safer by using the explicit size of the table rather than
an hypothetical maximum value given by the enum. Mostly because
in the past I edited a table and forgot to remove the corresponding
enum value.
ok stsp@
- Commit:
6a1434e3aa431b60168143564fbce8330b0f79c9
- From:
- Omar Polo <op@omarpolo.com>
- Date:
drop unnecessary zeroing after calloc()
- Commit:
723721e290eec4393f8cad56a6bb38ca9fd379c7
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: use 'more' for the tag listing too
prodded by stsp and jamsek; ok stsp@
- Commit:
4921c182c938f7e109ed3f516cc6249ff7d676bc
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: render less tags in the summary page
suggested / ok stsp
- Commit:
ac15152e219bed9c7f41007a39c3cbce83ea9386
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: render READMEs in the tree view
ok tracey@
- Commit:
7f65bb55b8cfe0f964fd7cb845acfbc28621730e
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: add patch action to serve diffs in plain text
ok tracey@
- Commit:
260fd73e179a3da7600820c158e13bf148dff25f
- From:
- Omar Polo <op@omarpolo.com>
- Date:
refactor gotweb_load_got_path; no functional change
ok tracey@
- Commit:
1632f50aca5cd94ed681c20fc18c2b8ab4857b9c
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
remove the gotwebd repository cache
It only had 4 slots so was never quite useful, and sharing of sock->pack_fds
across cached repositories seems problematic.
with help from + ok op@
- Commit:
7607b8e0588a18b371f96092b43969a53ac94b09
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
whitespace fix
- Commit:
26678adde25a1fb7fa3e4b78c82a888cc3b767e4
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: get rid of proc.[ch]
proc.c really shines when there's a network of different types of
processes, potentially with a various number of instances each,
that needs to exchange messages. Gotwebd instead has a much simpler
design, and using proc.c causes more overhead (/headaches) than it
solves.
So, this attempts to provide the same functionalities but with a
much simpler implementation that fits gotwebd better.
ok stsp@
- Commit:
03e70dd4d41906645718ee5b780d7e948404f292
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: improve gotwebd_assign_querystring()
This prevents the issues that we just hit with forgetting to remove
PREVID: instead of relying on the enum value to yield the size of
a table, use nitems(). Also, quit as soon as the matching key was
found, no need to iterate further.
ok stsp@
- Commit:
7781b9910fcbba579c2e112bbfdea73a64ccad70
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: render all the datetimes in a time tag
fixes an unused variable that should have been dropped in previous
commit too.
ok stsp@
- Commit:
bf26a633636ba2058b6bb747b0dd4ab17cb185a8
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: inline the only use of TM_RFC822
- Commit:
62eab86e6a1d5aea8a1bf90999c3c058b9aedd07
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: move the buffering from the fastcgi layer to the template
Reduces the indirection in fcgi.c, starts to make the struct template
opaque, simplifies the template usage. All with a net negative :-)
reads fine to stsp@ (thanks!)