These 2 new imports could be merged with all the others before into a single import statement. Yes/no?

Somebody has some ideas on some interesting config items to be exposed here?

Greenwave has only version but WaiverDB provides info on authentication method: https://waiverdb.engineering.redhat.com/api/v1.0/about

Maybe auth_backend makes sense for freshmaker. This can be: noauth or openidc

@lholecek I put auth_backend here..

+1

Code looks good for me. But I have a different thought about raising ProgrammingError from AboutAPI.get.

Let's see a case. An incorrect config is added to config_items, then freshmaker is released. When about API is called at some time after the release, client gets a bad request which represents the raised ProgrammingError. This shouldn't make sense. Instead, it would be good to catch the invalid config added to config_items during development. An assert could be appropriate for that.

@cqi, thanks for pointing it out, I used ValueError instead.

:+1: LGTM