feature: add redis resource access for PAM

[fangpenlin]

Description 📣

ref: https://linear.app/infisical/issue/PAM-42/add-redis-account-type-for-pam

depends on Infisical/infisical#5085

Type ✨

• Bug fix
• New feature
• Improvement
• Breaking change
• Documentation

Tests 🛠️

please see Infisical/infisical#5085

[greptile-apps]

Greptile Summary

This PR adds Redis resource access support to the PAM (Privileged Access Management) system, following the same architectural pattern as existing database, SSH, and Kubernetes integrations.

Key Changes

• Added Redis protocol support using custom resp3 library fork
• Implemented Redis connection proxy with AUTH credential injection
• Added comprehensive command relay handler supporting MONITOR mode, pub/sub, and PUSH messages
• Integrated Redis resource type into session logging and upload pipeline
• Added CLI commands: infisical pam redis access-account

Issues Found

• Critical logic error in authentication validation (packages/pam/handlers/redis/proxy.go:83) that will cause all Redis connections to fail

Confidence Score: 3/5

• This PR contains a critical authentication logic error that will prevent Redis connections from working
• The implementation follows established patterns and is well-structured, but contains a critical bug in packages/pam/handlers/redis/proxy.go:83 where the authentication check uses && instead of ||, which will cause all Redis authentication attempts to fail. Once this is fixed, the code should be safe to merge.
• packages/pam/handlers/redis/proxy.go requires immediate attention to fix the authentication logic error on line 83

Important Files Changed

Filename	Overview
packages/pam/handlers/redis/proxy.go	Redis proxy that authenticates to target server and handles client connections; AUTH logic has a potential issue
packages/pam/handlers/redis/relay_handler.go	Comprehensive Redis command relay with monitor mode, pubsub support, and session logging
packages/pam/local/redis-proxy.go	Local Redis proxy server with session management, approval flow, and graceful shutdown
packages/pam/pam-proxy.go	Added Redis resource type support to PAM proxy handler

[greptile-apps]

_{10 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}