PlanetTheCloud / project-logged

Custom material design authentication template for MyOwnFreeHost
MIT License
31 stars 17 forks source link

reCaptcha #25

Closed ImLoadingUuU closed 1 year ago

PlanetTheCloud commented 1 year ago

Too many unrelated changes. Please submit a new pull request with a 2.0 base as right now it looks like it still has leftovers from v1.8. Also do not add Unstable features please

ImLoadingUuU commented 1 year ago

ok very stable but master branch is 2.0

PlanetTheCloud commented 1 year ago

It looks like you're including the system config to $sysConfig, which is NOT recommended. Please use the config() function instead. You can do config('system.captcha.key') or something like that

ImLoadingUuU commented 1 year ago

done

4yx commented 1 year ago

Why did you close this? @ImLoadingUuU

ImLoadingUuU commented 1 year ago

Why did you close this? @ImLoadingUuU

i found a very weird bug.

the api return data is

{
  "success": true,
  "challenge_ts": "2023-04-30T13:11:21Z",
  "hostname": "localhost"
}{"status":"error","message":"Something went wrong, please try again later.","dev_details":{"exception":"Error","message":"Undefined constant \"success\"","file":"\/opt\/homebrew\/var\/www\/sys-auth\/api\/signup.php","line":35},"DEVELOPMENT_MODE":true}

theres 2 json, one is Google reCaptcha API,one is VistaPanel Create Account API. idk where is wrong with my code!

ImLoadingUuU commented 1 year ago

oh wait. after this reply i think i found the problem HostingAccount.php curl variable is same as reCaptcha Verification Part. they are $result too! (edit no.