PR#3404: Added information about authentication providers in default_config.py -

pagure

#3404 Added information about authentication providers in default_config.py

Merged Jul 10, 2018 by pingou. Opened Jul 10, 2018 by ymdatta.

ymdatta/pagure github_issue into master

ymdatta commented Jul 10, 2018

Came across this at https://github.com/pypingou/pagure/pull/21.
I thought it would be useful to give information regarding providers,
because i had a hard time understanding different providers when configuring
local_auth for my local pagure instance.

Signed-off by: Mohana Datta Yelugoti ymdatta@protonmail.com

pingou commented Jul 10, 2018

and enforces that users sign the FPCA

pingou commented Jul 10, 2018

Uses OpenID and defaults to the OpenID server from the Fedora Project, which can be changed using the configuration key...

pingou commented Jul 10, 2018

This is good but I think this is probably better to add in the documentation rather than the default_config.py at worst, we can mention the doc in the default_config.py.

ymdatta commented Jul 10, 2018

@pingou : I think that makes sense, will update it :)

rebased onto bc15e524352c2edfd2477189ad355cba8066d0fb

Jul 10, 2018

pingou commented Jul 10, 2018

Pretty please pagure-ci rebuild

pingou commented Jul 10, 2018

The trailing space is making the tests fail :)

pingou commented Jul 10, 2018

Commit 3df0e0ef fixes this pull-request

Pull-Request has been merged by pingou

Jul 10, 2018