Comment 4 for bug 1593192

Revision history for this message
Jamie Lennox (jamielennox) wrote :

This came up with a user recently - yes, we absolutely need this but you shouldn't have to fix the problem in load_from_opts. You should simply be able to put required=True on the plugin loader options that are required.

The most obvious one of these is access-token in OidcAccessToken, if you don't specify required here you get an error because you aren't passing enough options to the loader, but I'm suspecting that there are a number of other options in that file that also need to be marked.

Alvaro or Steve - you guys understand this better than I, can you go through the OIDC loaders and just put required=True next to required options.