Skip to content

Conversation

@javiercasares
Copy link
Owner

Summary

  • reuse the active Redis client exposed by WordPress object cache implementations when available
  • fall back to a temporary Redis connection configured via the WP_REDIS_* constants and close it after use
  • keep the CLI detection fallback while preventing Redis-related exceptions from bubbling up

Testing

  • vendor/bin/phpstan analyse --memory-limit=1G wpvulnerability-general.php
  • vendor/bin/phpunit (fails: missing /tmp/wordpress-tests-lib)
  • vendor/bin/phpcs (fails: missing PHPCompatibilityWP sniff)

https://chatgpt.com/codex/tasks/task_b_69045749c8d48321ac11ed4d420ae004

@amplify-security
Copy link
Contributor

amplify-security bot commented Oct 31, 2025

🔍 Amplify code check status:   status looks good

⚠️ 0 issues detected in   📄 1 file and   ❇️ 126 lines of code   🛠️ using Opengrep

Last updated by commit d1fea33 at 2025-10-31 06:53:43 UTC.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@sonarqubecloud
Copy link

@javiercasares javiercasares merged commit 6a82152 into dev Oct 31, 2025
3 checks passed
@javiercasares javiercasares deleted the devel/refactorizar-wpvulnerability_detect_redis-para-reutilizar-co branch October 31, 2025 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants