Skip to content

Conversation

@sadsfae
Copy link
Contributor

@sadsfae sadsfae commented Dec 18, 2025

  • Add verify parameter to control TLS certificate verification via @ cjeanner

grafuls and others added 6 commits November 10, 2025 19:09
Add a new 'verify' parameter to QuadsApi and QuadsBase constructors
to allow users to control TLS certificate verification behavior.

The parameter accepts:
- False (default): Disable certificate verification (backward compatible)
- True: Enable verification using system's default CA bundle
- str: Path to a custom CA bundle file

This change updates both the _make_request method and the login
method to use the verify parameter instead of hardcoded verify=False.

Changes:
- Add verify parameter to QuadsBase.__init__ with Union[bool, str] type
- Update _make_request to use self.verify
- Update login method to use self.verify
- Add comprehensive docstring explaining the parameter
- Add 6 new tests covering all verify parameter scenarios
- Update usage documentation with examples

Code generated with assistance from Auto (Cursor AI agent router).
All tests pass and maintain backward compatibility.
Add verify parameter to control TLS certificate verification
@sadsfae sadsfae changed the title Merge development latest Merge development to latest Dec 18, 2025
@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/quads_lib/base.py 100.00% <100.00%> (ø)
src/quads_lib/quads.py 99.64% <100.00%> (ø)
tests/test_quads.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sadsfae sadsfae merged commit 95c42a3 into latest Dec 18, 2025
15 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.

4 participants