| Age | Commit message (Collapse) | Author |
|---|
|
Redirect to login if not logged in, on login session cookie is set
and projects or reviews are listed.
|
|
|
|
|
|
|
|
|
|
|
|
Finally got around to fixing the pre-receive hook to include quarantined
objects so the hook actually can run git commands on the not-yet-accepted
commits. As part of that, had to make sure git hook and eyeballs server
had the same path to the repo or confusion will appear.
|
|
|
|
Reads from a list of grit files, all strings and their translations.
|
|
Add logs
Add remote keys for each user
|
|
Sets up a whole slew of docker instances, all started from clean
slate for test.
|
|
Only the owner or a maintainer of the project can remove a review.
Removing a review also removes the git branch.
Only reviews that are either draft or dropped can be removed.
|
|
But also add /review endpoint that takes an id. So you can get
review data with either the reviewid or the branchname.
|
|
Having to include --feature=build-server in basically all commands
that wasn't building eyeballs-githook got tiring quickly.
Instead, use workspaces, with a separate project for building
the githook. It means I also had to add a library common with
code shared by both githook and server.
|
|
|
|
Next step is to generate authorized_keys files for git server
based on keys.
|
|
Pushing a commit to a new branch creates a review.
Each project has its own git directory, with githooks installed
that talkes with server process via unix sockets.
|
|
Non existent projects, users and such.
|
|
User: username must be unique, use as primary key and drop id.
|
|
Preparing for git connection.
|
|
Non-test auth is now using ldap for account syncing and authentication.
Test auth is still using hardcoded users (user and other). But it is
now also possible to login as "other".
|
|
Use project_check_maintainer correctly, should not panic (by .unwrap)
but forward the error (by ?).
|
|
Also add /users endpoint.
|
|
Must be <version>_<description>.sql.
|
|
While doing that I realized I had forgotten to declare maintainers
for projects. Also added default roles and changed so that review_users
only contains overrides, so that changes to the project users is
instantly applied to all reviews (unless there is an override).
|
|
|
|
|
|
Let auth::stage() deal with having to clone the string
It shouldn't force the caller to create a String object.
|
|
|
|
The actual authentication is still fake.
|
|
Also move reviews to under project to make it obvious that projectid
is just that.
|
|
|
|
|
|
|
