2sic / app-mobius-forms

Simple jQuery based 2sxc form - to use immediately or to modify as needed. Multi-language, simple, with Recaptcha
MIT License
8 stars 4 forks source link

Fields missing even if everything is filled #55

Closed enfJoao closed 2 years ago

enfJoao commented 2 years ago

Even when all fields are filled, mobius says something is missing.

I hope all this testing and bug reporting earns me some patronage points :)

image

iJungleboy commented 2 years ago

@maaaximum-at-2sic can you reproduce this?

iJungleboy commented 2 years ago

@eXistenZe is this a standard form, or one you made?

maaaximum-at-2sic commented 2 years ago

Couldn't reproduce.

Tested using: Name Version
DNN 9.10.2
2sxc 14.01
Mobius Forms 5.04

Can you tell me more about your setup @eXistenZe ?

enfJoao commented 2 years ago

Same settings. Activated recaptcha and set the keys. The challenge does show. Then I get the missing field red info.

One warning on browser log: DevTools failed to load source map: Could not load content for https://woundcare.app/desktopmodules/tosic_sexycontent/dist/turnOn/turn-on.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE

maaaximum-at-2sic commented 2 years ago

@eXistenZe what version of reCAPTCHA do you use? Our implementation uses reCAPTCHA v3.

enfJoao commented 2 years ago

The guide says v2

On Fri, Jun 10, 2022, 11:40 Maxim Hüsler @.***> wrote:

@eXistenZe https://github.com/eXistenZe what version of reCAPTCHA do you use? Our implementation uses reCAPTCHA v3.

— Reply to this email directly, view it on GitHub https://github.com/2sic/app-mobius-forms/issues/55#issuecomment-1152226661, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGKKKX3YVNIMUESU22UXMH3VOMLTBANCNFSM5YLICS5A . You are receiving this because you were mentioned.Message ID: @.***>

maaaximum-at-2sic commented 2 years ago

I have updated the guide with the correct reCAPTCHA version (v3). Thank you @eXistenZe :smile:. Can you check if the bug is still present using reCAPTCHA v3?

enfJoao commented 2 years ago

Now it goes to the api but there is still an error: "Message":"Bad Request", "ExceptionMessage":"The remote server returned an error: (400) Bad Request.", "ExceptionType":"System.Net.WebException", "StackTrace":" at System.Net.WebClient.DownloadDataInternal(Uri address, WebRequest& request)\r\n at System.Net.WebClient.DownloadString(Uri address)\r\n at Recaptcha.Validate(String encodedResponse) in c:\inetpub\vhosts\x.x\x.x\Portals\0\2sxc\MobiusForms5\staging\api\Parts\Recaptcha.cs:line 25\r\n at CallSite.Target(Closure , CallSite , Object , String )\r\n at FormController.ProcessForm(Dictionary2 contactFormRequest, String workflowId) in c:\\inetpub\\vhosts\\x.x\\x.x\\Portals\\0\\2sxc\\MobiusForms5\\staging\\api\\FormController.cs:line 27\r\n at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ActionExecutor.<>c__DisplayClass6_1.<GetExecutor>b__0(Object instance, Object[] methodParameters)\r\n at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ExecuteAsync(HttpControllerContext controllerContext, IDictionary2 arguments, CancellationToken cancellationToken)\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Tracing.ITraceWriterExtensions.d17`1.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Controllers.ApiControllerActionInvoker.d1.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Tracing.ITraceWriterExtensions.d17`1.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Filters.ActionFilterAttribute.d6.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Web.Http.Filters.ActionFilterAttribute.d6.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Filters.ActionFilterAttribute.d5.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Controllers.ActionFilterResult.d5.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Filters.AuthorizationFilterAttribute.d3.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Filters.AuthorizationFilterAttribute.d3.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n at System.Web.Http.Controllers.ExceptionFilterResult.d6.MoveNext()" }

maaaximum-at-2sic commented 2 years ago

This error couldn't be reproduced and may be caused due to file modification or outside influence, such as a firewall. Please try installing Mobius-Forms on a new DNN installation with the newest 2sxc and new reCAPTCHA setup.

This project is open source, so feel free to create a pull request with your solution anytime. 😏

enfJoao commented 2 years ago

I apologize for the delay. I reinstalled everything and started all over, one step at a time. I got the errors all over the place again, even the missing recaptcha key fixed in the last update. One thing I've learned: this app is heavily dependent on clearing dnn cache after changing settings, specially recaptcha related. It seems to be working fine now. Thanks.

iJungleboy commented 2 years ago

@eXistenZe thanks for the update that it works now, and also the hint or dnn-cache. We were not aware of that.