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.

Sign up for GitHub

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

Jump to bottom

feat(csharp/src/Drivers/Apache/Spark): add option to ignore TLS/SSL certificate exceptions #4

Closed
wants to merge 65 commits into from
Closed

feat(csharp/src/Drivers/Apache/Spark): add option to ignore TLS/SSL certificate exceptions #4

wants to merge 65 commits into from

Conversation

birschick-bq
Copy link
Owner

Adds option to ignore TLS/SSL certificate errors.

Property Description Default
adbc.spark.tls_options Comma-separated list of TLS/SSL options. Each option indicates the TLS/SSL option when connecting to a Spark server.

Allowed values: allowinvalidcertificate, allowinvalidhostnames.

Option allowinvalidcertificate ignores certificate error due to missing certificate authority (typically when using a self-signed certificate). Option allowinvalidhostnames ignores certificate error due to a mismatch of the hostname. (e.g., when connecting through an SSH tunnel). Example adbc.spark.tls_options=allowinvalidcertificate

@birschick-bq
Add support for username and password authentication
7e189d1
@birschick-bq
Merge branch 'apache:main' into dev/birschick-bq/user-password-auth
95a3fd9
@birschick-bq
work-in-progress - support non-Arrow results version 3.3.1
e297113
@birschick-bq
make protocolVersion a parameter
e4c5f30
@birschick-bq
updates for HiveServer2Reader and test
586edcc
@birschick-bq
remove commented code
308a78f
@birschick-bq
small refactor/rename
f83c7a6
@birschick-bq
Merge remote-tracking branch 'origin/main' into dev/birschick-bq/spar…
e320463 
…k-ds-variants
@birschick-bq
refactor more GetRowSet
68ee762
@birschick-bq
case-insensitive compare
df54095
@birschick-bq
Merge branch 'apache:main' into dev/birschick-bq/user-password-auth
137caf7
@birschick-bq
improvements from code review comments
3e716b9
@birschick-bq
improvements from code review comments
c55abaf
@birschick-bq
Merge branch 'dev/birschick-bq/user-password-auth' into dev/birschick…
b23b6de 
…-bq/spark-ds-variants
@birschick-bq
Fix merge error
d359c78
@birschick-bq
Fixed compile and test issues
92490c7
@birschick-bq
Merge branch 'dev/birschick-bq/user-password-auth' into dev/birschick…
ba0a966 
…-bq/spark-ds-variants
@birschick-bq
Many more tests are passing
69131fa
@birschick-bq
Detect and handle null values in array columns.
4af848c
@birschick-bq
Detect and handle null values in array columns.
8e35f95
@birschick-bq
Add license.
ed56eb1
@birschick-bq
Improve clarity of tests with protocol/server versions.
828fd95
@birschick-bq
Fix test incompatibility.
c02241a
@birschick-bq
Add negative connection test for authentication and host.
bbe81a3
@birschick-bq
Merge branch 'dev/birschick-bq/user-password-auth' into dev/birschick…
33799d7 
…-bq/spark-ds-variants
@birschick-bq
Update documentation.
344cf2a
@birschick-bq
Refactor
67c14e2
@birschick-bq
Refactor
42365d7
@birschick-bq
correct project change
c57ed56
@birschick-bq
self code review #1
442d1c6
@birschick-bq
add variant specific validation
8c51e9c
@birschick-bq
invert compiler coditional
199e558
@birschick-bq
update documentation
0253b64
@birschick-bq
update for code review comments
0b3b6ab
@birschick-bq
update for code review comments #2
9c6f3ed
@birschick-bq
update for code review comments #3
3c5a4db
@birschick-bq
update for code review comments #4 - data type conversion option
8102d4b
@birschick-bq
feat(csharp/src/Drivers/Apache/Spark) Perform scalar data type conver…
b7658eb 
…sion for Spark over HTTP
@birschick-bq
add license text
66f4074
@birschick-bq
remove trailing space
5c42e26
@birschick-bq
some refactoring
98a4968
@birschick-bq
improve Impala example
44e8982
@birschick-bq
Merge remote-tracking branch 'origin/main' into dev/birschick-bq/data…
5b76cfc 
…-type-conv-scalar
@birschick-bq
Ensure data type conversion is a list of possible conversions.
43d59ee
@birschick-bq
Add test for data type conversion option parsing.
6fb9cf2
@birschick-bq
Merge remote-tracking branch 'origin/main' into dev/birschick-bq/data…
569d3b1 
…-type-conv-scalar
@birschick-bq
Corrected tests so that it handles Apache Spark 3.5
8bab697
@birschick-bq
Merge remote-tracking branch 'origin/main' into dev/birschick-bq/data…
a74cfe4 
…-type-conv-scalar
@birschick-bq
Merge remote-tracking branch 'origin/main' into dev/birschick-bq/cert…
1452a95 
…ificate-options
@birschick-bq
Changed enumeration to use flags
0ba07c6
@birschick-bq
Merge remote-tracking branch 'origin/main' into dev/birschick-bq/data…
2708aeb 
…-type-conv-scalar
@birschick-bq
Merge branch 'dev/birschick-bq/data-type-conv-scalar' into dev/birsch…
605c20b 
…ick-bq/certificate-options
@birschick-bq
feat(csharp/src/Drivers/Apache/Spark): add option to ignore TLS/SSL c…
8f149bf 
…ertificate exceptions
@birschick-bq
updated documentation
6889ba5
@birschick-bq
Merge branch 'main' into dev/birschick-bq/certificate-options
980b05c
@birschick-bq
Merge branch 'main' into dev/birschick-bq/certificate-options
7d3fa2e
@birschick-bq birschick-bq changed the base branch from dev/birschick-bq/data-type-conv-scalar to main September 25, 2024 18:32
birschick-bq added a commit that referenced this pull request Dec 12, 2024
@birschick-bq
remove more instrumentation code #4
41ed8e3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@birschick-bq