#5138 Add support for noggin with the FAS auth backend
Merged 2 years ago by ngompa. Opened 3 years ago by pingou.

file modified
+2
@@ -490,6 +490,8 @@ 

                  )

              ]

          groups = set(groups).union(admins)

+         if auth == "fas":

+             groups.add("signed_fpca")

          ext_committer = set(pagure_config.get("EXTERNAL_COMMITTER", {}))

          groups = set(groups).union(ext_committer)

          flask.g.unsafe_javascript = True

The particularity of this backend is to require that users sign the
Fedora Project Contributor Agreement. With the new Auth system in
Fedora, the contributors who have signed this agreement are in a
dedicated group while it was separated before. That does mean that
pagure needs to ask for the members of this group upon login,
otherwise it has no way to know if the user is a member of that group
(ie, if the user signed the agreement).

This commit fixes that.

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

rebased onto 262fabe02d5c0b203ec02e71fce284952b7fae0c

3 years ago

rebased onto 9aa24a69e1c134680d96c939967385e5c4de11bf

3 years ago

I love when dev/run-tests-container.py --centos --branch new_auth works locally but not on jenkins...

rebased onto 6a3f43d

2 years ago

Pull-Request has been merged by ngompa

2 years ago
Metadata