Skip to content
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

feature/add support for tls settings #93

Merged
merged 10 commits into from
Feb 16, 2024

Conversation

Nicoretti
Copy link
Member

@Nicoretti Nicoretti commented Feb 14, 2024

Relates to #exasol/ai-lab#196

  • Add pytest plugin to spin up local http server for testing
  • Add https and https settings tests for Service
  • Add support for verify argument
  • Fix ssl tests for service
  • Update documentation of Service constructor

@ahsimb
Copy link

ahsimb commented Feb 14, 2024

Looks good to me.

@exasol exasol deleted a comment from sonarqubecloud bot Feb 15, 2024
@exasol exasol deleted a comment from sonarqubecloud bot Feb 15, 2024
@Nicoretti Nicoretti requested a review from tkilias February 15, 2024 12:47
exasol/bucketfs/__init__.py Show resolved Hide resolved
exasol/bucketfs/__init__.py Show resolved Hide resolved
test/integration/bucketfs_test.py Outdated Show resolved Hide resolved
test/integration/bucketfs_test.py Show resolved Hide resolved
ckunki
ckunki previously approved these changes Feb 16, 2024
ahsimb
ahsimb previously approved these changes Feb 16, 2024
@Nicoretti Nicoretti dismissed stale reviews from ahsimb and ckunki via 80f386a February 16, 2024 09:16
@Nicoretti Nicoretti requested review from ckunki and ahsimb February 16, 2024 09:16
doc/conf.py Show resolved Hide resolved
@Nicoretti Nicoretti merged commit e16cc33 into main Feb 16, 2024
17 of 18 checks passed
@Nicoretti Nicoretti deleted the feature/add-support-for-tls-settings branch February 16, 2024 09:32
@exasol exasol deleted a comment from sonarqubecloud bot Feb 16, 2024
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