Implement configuration provider-based settings for the ReCaptcha module

[Piedone]

Is your feature request related to a problem? Please describe.

You currently need to configure ReCaptcha settings from settings. This is a good and necessary way to do it, but for environment-based settings (like different for staging and prod) it'd be better to have them configurable via configuration providers too. ReCaptchaSettings is almost ready for this.

Describe the solution you'd like

Similar to what we have in a lot of other modules:

• Provide an extension method to do environment configuration, similar to ConfigureAzureADSettings().
• Check that everywhere where ReCaptchaSettings is used, it's used as an IOptions<ReCaptchaSettings> and not directly as site settings (except when the site settings are edited). E.g. ReCaptchaLoginFilter directly goes to site settings instead of going the options way.
• Add corresponding docs.

This looks like a good first issue, since you just have to follow the pattern of existing implementations.

Describe alternatives you've considered

None.

[Piedone]

FYI @hishamco if you're interested and no one else takes on it.

[hishamco]

I saw it yesterday, but I was busy at that time .. hope to reply soon

[RAHULRR1]

is this issue resolved?

[hishamco]

I don't think so

[Harrish-Selvarajah]

Hi @Piedone, Can I take up this issue ?

[Piedone]

Sure, that would be great! Let me know when you're done and I can review.

[Harrish-Selvarajah]

Thank you! I hope I can use this thread to ask questions if I face any blockers.

[hishamco]

@Harrish-Selvarajah I assigned this to you

[Piedone]

Will you work on this any time soon, @Harrish-Selvarajah?

[Harrish-Selvarajah]

Hey @Piedone , Work has been a bit hectic the past weeks. I'm planning to put some time this weekend.

[Piedone]

OK, thanks!

[Ankur-Thakur-NEU]

Hi @Piedone, I'm following up on the reported issue. Could you please confirm if this enhancement is done, or if there have been any updates or resolutions since your last report?

[Piedone]

There hasn't been any work done on this to my knowledge.

[Harrish-Selvarajah]

Appologies, I couldn't find the time to work on this problem. If anyone is willing to contribute their time, please do!

[Ankur-Thakur-NEU]

I'll take the ticket and raise a pr soon. Thank you.

[meijiro-yumeru]

Hello. Is anyone working with this issue?

[hishamco]

@Harrish-Selvarajah was planning to take it, but not

@meijiro-yumeru feel free to take it

[meijiro-yumeru]

I’ll see what I can do, I’m just asking just in case if I could give it a try.

[meijiro-yumeru]

The implementation would involve using Envrionment variable via a web service(i.e. using Amazon Aws, Google Cloud Provider, etc)?

[Piedone]

While the configs might ultimately come from an env var, you mustn't read those directly. This issue is about reading values from IShellConfiguration and use the ASP.NET Core configuration provider concept. See https://docs.orchardcore.net/en/latest/reference/modules/Configuration/.