Commit Briefs

4429bcbe67 Thomas Adam

portable: release 0.114 (tags/0.114)


1873f8e9ab Thomas Adam

bump versio number


2ff12d2abc Thomas Adam

CHANGES for 0.114



01b6b6f8d2 Thomas Adam

avoid event_loopexit() in gotsysd libexec event handlers when flushing

It seems flushing our final message can fail if we forcefully exit the event loop here. This would leave gotsysd's sysconf process stuck in a state where it was waiting for a message which never arrived.


05a4f4a0af Thomas Adam

use imsgbuf_flush() instead of gotsysd_imsg_flush() in libexec event handlers

The gotsysd_imsg_flush() wrapper enforces a sync write by polling the file descriptor and writing data in a loop. Since libevent already does polling for us we can simply use imsgbuf_flush() here instead.




4f6bf13c6b Thomas Adam

always consume authorized keys messages in gotsys-userkeys

THe main process will always send these messages so don't skip parsing them even if we are going to ignore the current user.



40d558910d Thomas Adam

stop running ssh with -q by default

The -q option suppresses too many important errors, including the warning printed when a host key has changed. Requiring users to pass -v in order to see this message is too cumbersome.


89dafad3e9 Thomas Adam

build and install gotwebd in gotsysd regress VMs



dd1da84ce9 Thomas Adam

untangle committer and author_time parameters of got_worktree_commit()

Make the semantics of the author_time parameter self-contained by treating author_time == 0 as the fallback case, instead of treating committer == NULL as the fallback case. Having the semantics of one parameter depend on the semantics of another might lead to subtle bugs introduced during potential future refactoring of this code.


8acf3aa649 Thomas Adam

cvg: add author_time argument to got_worktree_cvg_commit()

No functional change, but keeps the code in sync with got. Patch by Sven M. Hallberg


e16f7949b8 Thomas Adam

got rebase: preserve author timestamps

Adds test_rebase_preserves_author_data to regress/cmdline/rebase.sh. Patch by Sven M. Hallberg


4071bc439f Thomas Adam

relax repository path permission checks in gotsys-repo-create

Since gotwebd can now be given read access to gotd repositories by making the directory readable for the _gotwebd group, these checks are too strict. Stop requiring a specific GID, and allow group rx bits to be set for group read access. As before, keep rejecting world/other rwx bits for safety. Read access to repositories is supposed to be managed by gotd.


ddb085b36d Thomas Adam

fix gotsys-read-conf crash when ssh key comments are missing in gotsys.conf

Add test coverage for this issue.


9f4ec8bd2a Thomas Adam

bump version number


d029e8951a Thomas Adam

CHANGES for 0.113


41e722a198 Thomas Adam

fix a race in gotd notification processing

Dequeue notifications from the list only once the notify process has confirmed that the notification has been sent. This should be more robust for the following reason: In some spots were are checking the list head against NULL to see if any notifications are still pending. By removing notifications from this list too early, this check did not cover notifications which were still in flight. We could thus end up deciding to shut down the session process too early, and the last notification of several would fail. Issue observed when sending a new commit on a branch and a new tag which tags this commit to gotd in a single 'got send' operation. Only the tag notification would make it, while the other notification never arrived after an 'unexpected EOF' error in the notify process while trying to send a confirmation to the session process. ok op@


689eb4525d Thomas Adam

fix bogus "unexpected privsep message" from gotsh during "got send"

Ensure that gotsh receives its end of the pack file data pipe before repo_write starts sending pack file status messages. Messages of type GOTD_IMSG_PACKFILE_StATUS would end up being received in gotsh's serve_write() function too early. This race is similar to the one fixed for repo_read back in commit c2274a511a7415078e2628f969b8459f69432411 Reported by martijn@, who pin-pointed the problematic case in the code, thanks!



818df7b507 Thomas Adam

portable: set next version


2d26ff5622 Thomas Adam

portable: release 0.113 (tags/0.113)