Commit Briefs

Thomas Adam

gotd: don't ignore error in session_dispatch_repo_child

ok stsp


Thomas Adam

got-notify-http: implement basic authentication

ok stsp



Thomas Adam

Remove some dead code


Thomas Adam

Rewrite got-fetch-http I/O using bufio API. This should help make the

code more portable supports almost everything we need out of the box. pledge needs the rpath promise because of libtls. This was technically also the case with the old API but we didn't actually enable pledge thanks to DEBUG_HTTP. The synchronous bufio wrappers should probably move to lib/bufio.c in the future. ok and feedback from op@



Thomas Adam

got-notify-http: add the repository name in the json

ok stsp


Thomas Adam

Unset errno and bio->wantev if error was not TLS_WANT_*. This is commonly

called in a loop, if we don't clear them we might keep on looping. ok stsp@




Thomas Adam

Rename got-http helper to got-fetch-http to match naming scheme.

We aren't planning to support sending anyway. ok stsp@


Thomas Adam

Add initial read-only http fetch support using a got-http helper.

Currently we only support the smart protocol with a limited feature set. ok stsp@ tobhe@



Stefan Sperling

drop privileges in all parts of gotd other than the main process

ok thomas_adam


Thomas Adam

remove outdated TOOD item and tweak test accordingly

The default deny policy is working as expected according to newly added regression test. Tweak this test to use an implicit read-only rule, rather than an explicit one, as stated in the TODO item.



Thomas Adam

portable: README: mention jot's removal

Upstream has now moved from jot -> seq.


Thomas Adam

fix gotd notification test failures due to missing shell quoting

The expected output generated by test scripts was wrong on days with a single-digit date. Found by Omar's regress builder. ok op@


Thomas Adam

regress: use seq instead of jot

This is just for -portable since jot is not always available. ok stsp@



Thomas Adam

remove trailing whitespaces


Thomas Adam

got.1: escape Eq since it's a GNU roff macro

`.Op Fl Eq' gets rendered as [-] with groff because Eq it's a macro. Escape it to get it rendered correctly. Discussed with bentley. Reported by casaca on the IRC channel, thanks!


Thomas Adam

initial documentation for the got-notify-http json payload

committing on behalf of op@, ok by me


Thomas Adam

portable: netbsd: add stdarg for enter_chroot

stdarg.h is required on NetBSD for variadic definitions.