Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@aschlean
Copy link
Contributor

@aschlean aschlean commented Oct 14, 2025

Pull Request Template

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Security enhancement
  • Performance improvement
  • Code refactoring (no functional changes)

Note

Replaces manual MCP session setup with shared MCPClientManager for OAuth-aware connections in security and compliance testers, updating connect/disconnect and OAuth test flows.

  • Testing Framework:
    • OAuth-aware connection management:
      • Replace manual streamablehttp_client/ClientSession.initialize() flow with MCPClientManager in security/security_tester.py and testing/compliance/mcp_compliance_tester.py.
      • Maintain backward-compatible session reference; track server_id per connection.
      • Update disconnect to call MCPClientManager.disconnect_server and clear server_id.
    • Security tests (security_tester.py):
      • _connect_to_server and _connect_with_oauth now use MCPClientManager for OAuth handling; temporary test connections are created/cleaned for token checks.
    • Compliance tests (mcp_compliance_tester.py):
      • _connect_to_server now uses MCPClientManager with SDK availability checks; improved error handling and session cleanup.

Written by Cursor Bugbot for commit a69cd5a. This will update automatically on new commits. Configure here.

wbbw1 and others added 4 commits October 13, 2025 18:23
added oauth for compliance and security tests
…mplemented and verified for Security and Compliance testers, consolidating their connection logic under MCPClientManager.
@aschlean aschlean self-assigned this Oct 14, 2025
@aschlean aschlean merged commit c15e3b0 into main Oct 14, 2025
3 checks passed
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.

2 participants