Update config parameters to match latest OIDC release and fix typos. …

[uoe-pjackson]

Summary

Update Apache::OIDCSettings parameters to match latest mod_auth_openidc release.

• Add parameters previously not included
• Fix typos in paramater names
• Update exisiting datatypes to match updated OIDC usage

Related Issues (if any)

Fixes #2567 #2566

[smortex]

Nice work, and good catch for the typos for PKCDMethod and UserInfoSignedResposeAlg.

I added some in-line comments with suggestions to fix some minor issues. Feel free to amend your commit, and to reply if something is not clear for you.

Thank you!

[uoe-pjackson]

@smortex Thanks for you comments. I have made an extra commit that addresses Integer, String and Apache::OnOff data-types in the rest of the file.

[smortex]

Good job. There are still 2 open comments where I think it is worth improving. I was not providing many details, so added more context of the reason why I am suggesting these changes.

Feel free to review them and tell me if you spot something that seems wrong.

Thank you!

[uoe-pjackson]

@smortex
Woops, I had overlooked those two suggestions. I've updated as you suggested. Thanks for you patience!

[smortex]

That looks awesome! Thank you for your work on this.

This looks complete to me, but I'll let other people review this PR and merge it if they feel it's ready.

[pebtron]

Thanks for this! There are typos in both REFERENCE.md and oidcsettings.php at the line:
Optional['SessionCookieChunkSize'] => Intege[-1],

[gcoxmoz]

FYI, this introduces a small breakage. UserInfoRefreshInterval went from Integer to Pattern and so we have to quote integer as strings.