Comparing changes

## Changes Support custom AuthVisitors ## Tests - [X] `make test` passing - [X] `make fmt` applied - [X] relevant integration tests applied

…#864) ## Changes Most services use `RESOURCE_DOES_NOT_EXIST` error code with 404 status code to indicate that a resource doesn't exist. However, for legacy reasons, Jobs and Clusters services use `INVALID_PARAMETER_VALUE` error code with 400 status code instead. This makes tools like Terraform and UCX difficult to maintain, as these services need different error handling logic. However, we can't change these behaviors as customers already depend on the raw HTTP response status & contents. This PR corrects these errors in the SDK itself. SDK users can then do ```go _, err := w.Jobs.Get(ctx, jobs.GetJobRequest{JobId: 12345}) if errors.Is(err, apierr.ErrResourceDoesNotExist) { ... } ``` just as you would expect from other resources. Updated the README with more information about this as well. ## Tests Added unit tests for error overrides. Added 2 integration tests for Clusters and Jobs. - [x] `make test` passing - [x] `make fmt` applied - [x] relevant integration tests applied

## Changes Besides the `Authorization` token, there are other tokens included by the SDK in requests in Azure & GCP with headers `X-Databricks-Azure-SP-Management-Token` and `X-Databricks-GCP-SA-Access-Token`. These need to be redacted as well unless DebugAuthorizationHeaders is set. ## Tests  - [x] `make test` passing - [x] `make fmt` applied - [ ] relevant integration tests applied

## Changes Because the debug string for requests/responses can contain percent characters, simply logging the request/response causes Go to interpret this as a format string. Instead, we should use format string `"%s"` to ensure that the percents are printed as expected to the debug log. ## Tests  - [ ] `make test` passing - [ ] `make fmt` applied - [ ] relevant integration tests applied

## Changes Added clientId and clientSecret to oauth-m2m auth_types Fixes these 2 properties being incorrectly marked as not used in oauth-m2m authnetication. ## Tests  - [x] `make test` passing - [x] `make fmt` applied - [x] relevant integration tests applied

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comparing changes

Open a pull request

Commits on Apr 3, 2024

Commits on Apr 5, 2024

Commits on Apr 8, 2024

Commits on Apr 11, 2024

This comparison is taking too long to generate.

Uh oh!