Skip to content

INTERNAL: Run a replication cluster with the extension in CI - #7

Merged
ing-eoking merged 1 commit into
developfrom
ci-cluster
Oct 24, 2026
Merged

ing-eoking merged 1 commit into
developfrom
ci-cluster

Conversation

@ing-eoking

Copy link
Copy Markdown
Collaborator

🔗 Related Issue

⌨️ What I did

CI의 테스트 코드 실행을 위한 환경 구성 및 보안 취약점 개선에 관한 내용으로, 우선 반영하겠습니다.
복제 환경을 포함한 CI 환경 구성 변경 사항은 추후 테스트 코드를 통해 검증할 수 있습니다.

주요 변경 사항은 다음과 같습니다.

  • CI 환경 개선: 복제 환경에서도 테스트를 수행할 수 있도록 CI 환경을 구성했습니다.
  • 보안 취약점 개선:
    • include/memcached/*.h 파일이 artifact에 노출되던 문제 수정
    • URL에 토큰을 삽입하여 발생할 수 있는 토큰 노출 위험 개선
    • 컨텍스트 변수를 직접 삽입하여 발생할 수 있는 Command Injection 취약점 개선
  • 오류 처리 개선: 파이프라인의 에러 처리 로직을 보완하여 잘못된 실패 판정이 발생할 가능성을 줄였습니다.

@coderabbitai

coderabbitai Bot commented Oct 24, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ing-eoking
ing-eoking merged commit 5e6b1af into develop Oct 24, 2026
3 of 4 checks passed
@ing-eoking
ing-eoking deleted the ci-cluster branch October 24, 2026 13:40

This branch was previously deployed

1 inactive deployment
ee — d552ba66 Deployed Oct 24, 2026 by ing-eoking via Build arcus-memcached #20
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