Hey Geoff,

Check out this topic https://our.umbraco.org/forum/umbraco-pro/contour/63765-Upgrading-ReCaptcha-to-the-Im-not-a-robot-version

It has all the details needed in order to upgrade

Hi Tim, does this work with Umbraco Forms or just Contour?

I've been trying to implement this using the method specified but both "FormViewEventArgs" and "FormRenderController" don't appear to exist.

thanks

I've got it working!

Pretty much same as the linked thread except I swapped out the references of "FormViewEventArgs" to use "FormValidateEventArgs" and swapped "FormRenderController" for "UmbracoFormsController".

In the FieldType.Recaptcha view I removed the "theme" setting as well, since none of the options apply to Recaptcha 2.0. Only "dark" and "light" are applicable, which aren't available in UF.

I can provide exact code that I used if anyone needs it but it's pretty much the same with those substitutions.

OK cool.. here we go..

First after you've registered your domain with Google's Recaptcha service, you need this tag in your "head" tag:

<script src='https://www.google.com/recaptcha/api.js'></script>

Then on your application started event (in this case I'm using WebActivatorEX for this so I've got an App_Start directory, but otherwise could be in a Global.asax or anywhere else I suppose:

UmbracoFormsController.FormValidate += UFEvents.FormValidate;

Then in my "UFEvents" class I've got this:

public static class UFEvents
{
    public static void FormValidate(object sender, FormValidationEventArgs e)
{
    var reCaptchaField = e.Form.AllFields
        .SingleOrDefault(f => f.FieldType.Name == "Recaptcha");
    if (reCaptchaField != null)
    {
        var httpContext = HttpContext.Current;
        var reCaptchaResponse = httpContext.Request["g-recaptcha-response"];
        bool isValid = IsReCaptchaValid(reCaptchaResponse, new HttpRequestWrapper(httpContext.Request));

        if (!isValid)
        {
            var controller = sender as Controller;
            controller.ModelState.AddModelError(reCaptchaField.Id.ToString(), "There was a problem verifying your request, please try again.");
        }
    }
}
public static bool IsReCaptchaValid(string gResponse, HttpRequestBase request)
    {
        bool isSuccess = false;

        using (var client = new WebClient())
        {
            var values = new NameValueCollection();
            values["secret"] = ConfigurationManager.AppSettings["ReCaptchaSecretKey"];
            values["response"] = gResponse;
            values["remoteip"] = GetIpAddress(request);

            var response = client.UploadValues(ConfigurationManager.AppSettings["ReCaptchaVerificationUrl"], values);
            var responseString = Encoding.Default.GetString(response);

            if (!string.IsNullOrEmpty(responseString))
            {
                JObject o = JObject.Parse(responseString);
                string successStr = (string)o["success"];
                bool.TryParse(successStr, out isSuccess);
            }
        }

        return isSuccess;
    }

private static string GetIpAddress(HttpRequestBase request)
    {
        // Look for a proxy address first
        var ip = request.ServerVariables["HTTP_X_FORWARDED_FOR"];

        // If there is no proxy, get the standard remote address
        if (string.IsNullOrWhiteSpace(ip)
            || string.Equals(ip, "unknown", StringComparison.OrdinalIgnoreCase))
            ip = request.ServerVariables["REMOTE_ADDR"];
        else
        {
            //extract first IP
            var index = ip.IndexOf(',');
            if (index > 0)
                ip = ip.Substring(0, index);

            //remove port
            index = ip.IndexOf(':');
            if (index > 0)
                ip = ip.Substring(0, index);
        }

        return ip;
    }
}

It's worth noting that I've got the Recapthca client key and secret key as Appsettings in my web.config here instead of using the ones in the forms.config file because I'm using this same code for another non-umbraco-forms form.

Finally I've just replaced the FieldType.Recaptcha.cshtml file with this:

@model Umbraco.Forms.Mvc.Models.FieldViewModel
@{
    var siteKey = ConfigurationManager.AppSettings["ReCaptchaSiteKey"];
}
 @if (!string.IsNullOrEmpty(siteKey))
{   
    <div class="g-recaptcha" data-sitekey="@siteKey"></div>
}

As you'll see I just included the recaptcha div tag with the key from my web.config. I'm sure the server generated tag that was there originally might have included the script tag as well but I was using recaptcha somewhere else so just did it like this. Also didn't bother with themes since Recaptcha 2 just has light and dark and I was happy with the default of light.

Hi Geoff,

i´m new in umbraco system and even not so skilled in .NET, but i´m using Umbraco Forms with reCaptcha and need to upgrade it.

In Global.asax i have only one line

<%@ Application Inherits="Umbraco.Web.UmbracoApplication" Language="C#" %>

so should i insert UmbracoFormsController.FormValidate += UFEvents.FormValidate; here?

and your class UFEvents is in App_Code folder or somewhere else?

Thanks

Hi Zedna,

Are you using Visual Studio? Are you comfortable with building/compiling code with it?

Basically you need to add a codebehind file to the Global.asax in order to add that code to the Application Start event.. but the instructions are slightly different depending on if you're using VS and rebuilding the code or not.

If you let me know I can give more specific instructions

To add code to the Application Started event in your Global.asax file you need to add a codebehind file for it.

First, in Visual Studio, add a new class to the project called Global.asax.cs at the root of the site (whatever directory Global.asax is in)

In that file, replace the code block starting with "public class Global" with this instead:

public class CustomGlobal : UmbracoApplication
{
    protected override void OnApplicationStarted(object sender, EventArgs e)
    {
        base.OnApplicationStarted(sender, e);

        UmbracoFormsController.FormValidate += UFEvents.FormValidate; 
    }
}

If you try to build the project right now you will have an error saying that UFEvents doesn't exist or something like that. Right click the word "UFEvents" in the Global.asax.cs file code and click "Resolove" then click on the appropriate "using" statement that comes up in this menu. You should now be able to build.

Save this file, and then right click "Global.asax" and click "View Markup". Here, you need to add the "CodeBehind" attribute and edit the "Inherits" attribute like so:

<%@ Application CodeBehind="Global.asax.cs" Inherits="umb_test.CustomGlobal" Language="C#" %>

Here, change the "umb_test" bit to match whatever the namespace of your Global.asax.cs file is. Whatever comes after "namespace" in that file.

Build the project and run and that code should now get hit on the (only) on first time the app runs... might help to put a breakpoint in the code while running to see if it's actually getting hit. If you want to try it again, you can effectifely restart the app by touching the web.config file. (just make a tiny change like adding a space somewhere and saving it. This will force IIS to restart the app next time the website is accessed.

Hi again, you'll need to add the following to the top of your Global.asax.cs file:

using Umbraco.Forms.Web.Controllers;
using Umbraco.Web;

Also, I think you might need to create your UFEvents class in a different folder to the "App_Code" folder. Once you do that you'll also need to add another "using" to global.asax.cs to reference the namespace of UFEvents.

is working on a reply...