search

Azure / azure-kusto-spark

Apache Spark Connector for Azure Kusto
Apache License 2.0
77 stars 34 forks source link

SAS URL regex pattern fix #299

Closed asaharn closed 1 year ago

asaharn commented 1 year ago

Pull Request Description

Fixes sas url parsing for invalid urls

Future Release Comment

[Add description of your change, to include in the next release] [Delete any or all irrelevant sections, e.g. if your change does not warrant a release comment at all]

Breaking Changes:

Features:

Fixes:

ohadbitt commented 1 year ago

KustoSourceE2E has a TODO to test this . I guess you tested it yourself but consider implementing the TODO // val storage = new TransientStorageParameters(Array(new TransientStorageCredentials(blobSas))) . . . // KustoSourceOptions.KUSTO_TRANSIENT_STORAGE -> storage.toString,