Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CaptchaRequest to merge CAPTCHA functionality and fix related code for retrieving QQ information in the comment section and unclickable image upload button. #320

Open
wants to merge 11 commits into
base: 3.x
Choose a base branch
from

Conversation

smallxu038
Copy link

I added a common captcha to the comment section. The captcha generates a token on the front end, which is submitted along with the comment form. In ‘inc/theme_plus.php‘, the token’s validity is verified with the CAPTCHA server. The old QQ information API could no longer be used, so I replaced it with another available API. I also resolved the issue of the image upload button being unclickable.
Because I’m not proficient in JavaScript, I’m unsure if merging the JavaScript for reCAPTCHA-v2 invisible into sakura-app.js would cause other issues. Therefore, I have kept it in footer.php.
cloudflare
reCAPTCHAV2
reCAPTCHAV3
mCAPTCHA
Upload images

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant