#3689 Add keyhelper and aclchecker
Merged by pingou. Opened by puiterwijk.
puiterwijk/pagure keyhelpers  into  master

Download 3689.patch

These scripts together can be used to replace the Authorized Keys part and read ACL parts of gitolite.

Note that the keyhelper is very naive for now with key lookups, details are in the commit (message).

rebased onto 22e4dfe4b9c19ce0edc70ae70fd256d63a34ff67

rebased onto 7db934b4d67d7ad4371100c7bd11fbe5a335e23f

Elsewhere we've used:

if "PAGURE_CONFIG" not in os.environ and os.path.exists(
    "/etc/pagure/pagure.cfg"
):
    os.environ["PAGURE_CONFIG"] = "/etc/pagure/pagure.cfg"

If repo is main, shouldn't we check that the user is allowed to push to it?

We should add these two to the doc/configuration.rst

This one also needs something in the doc

Please adjust the packaging so that the shebangs are going to get rewritten properly.

No, this script only checks whether the person has any access to the repo to clone.
For push, we depend on the dynamic ACLs, since those have significantly more information available.

rebased onto 7b7c2388510c68277fc0069685fa2cf48f1d5d45

1 new commit added

  • Add keyhelper and aclchecker to spec

Pull-Request has been merged by pingou

Metadata