Skip to content

Add X509 authentication tests. #1771

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

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Conversation

vbabanin
Copy link
Member

@vbabanin vbabanin self-assigned this Jul 20, 2025
@vbabanin vbabanin requested a review from Copilot July 20, 2025 20:24
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds comprehensive X.509 authentication tests for the MongoDB Java driver. The implementation introduces functional tests that validate X.509 certificate-based authentication across both synchronous and reactive drivers.

  • Implements abstract test framework for X.509 authentication with parameterized SSL configurations
  • Adds test implementations for both sync and reactive stream drivers
  • Integrates X.509 authentication tests into the CI/CD pipeline with Evergreen configuration

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
AbstractX509AuthenticationTest.java Core test logic with SSL context setup and parameterized test methods
X509AuthenticationTest.java (sync) Sync driver implementation extending the abstract test class
X509AuthenticationTest.java (reactive) Reactive streams implementation with sync adapter wrapper
run-x509-auth-tests.sh Shell script for keystore provisioning and test execution
.evg.yml Evergreen configuration additions for CI integration

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