Merge branch 'main' into oas-bot-23836109914/iaas

Author: cgoetz-inovex

CHANGELOG.md

@@ -22,6 +22,11 @@
 - `git`: [v0.11.1](services/git/CHANGELOG.md#v0111)
   - **Dependencies:** Bump STACKIT SDK core module from `v0.23.0` to `v0.24.0`
 - `iaas`:
+  - [v1.9.0](services/iaas/CHANGELOG.md#v190)
+    - **Feature:** Add `Cascade` field to `ApiDeleteVolumeRequest` model
+    - **Feature:** Add `ConfigDrive` field to `CreateServerPayload` and `Server` model
+  - [v1.8.2](services/iaas/CHANGELOG.md#v182)
+    - **Dependencies:** Bump SDK resourcemanager module from `v0.21.0` to `v0.21.1`
   - [v1.8.1](services/iaas/CHANGELOG.md#v181)
     - **Dependencies:** Bump STACKIT SDK core module from `v0.23.0` to `v0.24.0`
   - [v1.8.0](services/iaas/CHANGELOG.md#v180)
@@ -74,8 +79,11 @@
   - **Dependencies:** Bump STACKIT SDK core module from `v0.23.0` to `v0.24.0`
 - `serverupdate`: [v1.4.2](services/serverupdate/CHANGELOG.md#v142)
   - **Dependencies:** Bump STACKIT SDK core module from `v0.23.0` to `v0.24.0`
-- `serviceaccount`: [v0.15.1](services/serviceaccount/CHANGELOG.md#v0151)
-  - **Dependencies:** Bump STACKIT SDK core module from `v0.23.0` to `v0.24.0`
+- `serviceaccount`:
+  - [v0.16.0](services/serviceaccount/CHANGELOG.md#v0160)
+    - **Feature:** Add `TokenEndpoint` field to `ServiceAccount` model
+  - [v0.15.1](services/serviceaccount/CHANGELOG.md#v0151)
+    - **Dependencies:** Bump STACKIT SDK core module from `v0.23.0` to `v0.24.0`
 - `serviceenablement`: [v1.4.2](services/serviceenablement/CHANGELOG.md#v142)
   - **Dependencies:** Bump STACKIT SDK core module from `v0.23.0` to `v0.24.0`
 - `sfs`: [v0.6.2](services/sfs/CHANGELOG.md#v062)
@@ -99,9 +107,11 @@
   - **Dependencies:** Bump STACKIT SDK core module from `v0.21.1` to `v0.24.0`
 - `vpn`: [v0.4.1](services/vpn/CHANGELOG.md#v041)
   - **Dependencies:** Bump STACKIT SDK core module from `v0.23.0` to `v0.24.0`
-
-- `core`: [v0.24.0](core/CHANGELOG.md#v0240) 
-  - **Bugfix:** Allow setting waiter timeouts via context, that are longer than the default timeout.
+- `core`:
+  - [v0.24.1](core/CHANGELOG.md#v0241)
+    - **Improvement:** Fix misleading error messages in authentication setup and credentials parsing.
+  - [v0.24.0](core/CHANGELOG.md#v0240)
+    - **Bugfix:** Allow setting waiter timeouts via context, that are longer than the default timeout.
 
 ## Release (2026-03-27)
 - `alb`: 

core/CHANGELOG.md

@@ -1,3 +1,6 @@
+## v0.24.1
+- **Improvement:** Fix misleading error messages in authentication setup and credentials parsing.
+
 ## v0.24.0
 - **Bugfix:** Allow setting waiter timeouts via context, that are longer than the default timeout.
 

core/VERSION

@@ -1 +1 @@
-v0.24.0
+v0.24.1

core/auth/auth.go

@@ -54,7 +54,7 @@ func SetupAuth(cfg *config.Configuration) (rt http.RoundTripper, err error) {
 	} else if cfg.WorkloadIdentityFederation {
 		wifRoundTripper, err := WorkloadIdentityFederationAuth(cfg)
 		if err != nil {
-			return nil, fmt.Errorf("configuring no auth client: %w", err)
+			return nil, fmt.Errorf("configuring workload identity federation client: %w", err)
 		}
 		return wifRoundTripper, nil
 	} else if cfg.ServiceAccountKey != "" || cfg.ServiceAccountKeyPath != "" {
@@ -278,7 +278,7 @@ func readCredentialsFile(path string) (*Credentials, error) {
 	var credentials Credentials
 	err = json.Unmarshal(credentialsRaw, &credentials)
 	if err != nil {
-		return nil, fmt.Errorf("unmaPrivateKeyrshalling credentials: %w", err)
+		return nil, fmt.Errorf("unmarshalling credentials: %w", err)
 	}
 	return &credentials, nil
 }

core/auth/auth_test.go

@@ -367,6 +367,18 @@ func TestReadCredentials(t *testing.T) {
 	}
 }
 
+func TestReadCredentialsFileErrorMessage(t *testing.T) {
+	setTemporaryHome(t)
+
+	_, err := readCredentialsFile("test_resources/test_invalid_structure.json")
+	if err == nil {
+		t.Fatalf("error expected")
+	}
+	if !strings.Contains(err.Error(), "unmarshalling credentials") {
+		t.Fatalf("expected unmarshalling credentials error, got %s", err)
+	}
+}
+
 func TestDefaultAuth(t *testing.T) {
 	privateKey, err := generatePrivateKey()
 	if err != nil {
@@ -768,6 +780,23 @@ func TestKeyAuthPemInsteadOfJsonKeyErrorHandling(t *testing.T) {
 	}
 }
 
+func TestSetupAuthWorkloadIdentityErrorMessage(t *testing.T) {
+	setTemporaryHome(t)
+	t.Setenv("STACKIT_SERVICE_ACCOUNT_EMAIL", "")
+	t.Setenv("STACKIT_FEDERATED_TOKEN_FILE", "")
+
+	_, err := SetupAuth(&config.Configuration{WorkloadIdentityFederation: true})
+	if err == nil {
+		t.Fatalf("error expected")
+	}
+	if !strings.Contains(err.Error(), "configuring workload identity federation client") {
+		t.Fatalf("expected workload identity federation error, got %s", err)
+	}
+	if strings.Contains(err.Error(), "configuring no auth client") {
+		t.Fatalf("unexpected no auth error message: %s", err)
+	}
+}
+
 func TestNoAuth(t *testing.T) {
 	for _, test := range []struct {
 		desc string

examples/auditlog/go.mod

@@ -7,7 +7,7 @@ replace github.com/stackitcloud/stackit-sdk-go/services/auditlog => ../../servic
 
 require (
 	github.com/stackitcloud/stackit-sdk-go/core v0.24.0
-	github.com/stackitcloud/stackit-sdk-go/services/auditlog v0.1.5
+	github.com/stackitcloud/stackit-sdk-go/services/auditlog v0.3.2
 )
 
 require (

examples/authentication/go.mod

@@ -7,7 +7,7 @@ replace github.com/stackitcloud/stackit-sdk-go/services/dns => ../../services/dn
 
 require (
 	github.com/stackitcloud/stackit-sdk-go/core v0.24.0
-	github.com/stackitcloud/stackit-sdk-go/services/dns v0.17.6
+	github.com/stackitcloud/stackit-sdk-go/services/dns v0.19.2
 )
 
 require (

examples/authorization/go.mod

@@ -5,7 +5,7 @@ go 1.21
 // This is not needed in production. This is only here to point the golangci linter to the local version instead of the last release on GitHub.
 replace github.com/stackitcloud/stackit-sdk-go/services/authorization => ../../services/authorization
 
-require github.com/stackitcloud/stackit-sdk-go/services/authorization v0.12.0
+require github.com/stackitcloud/stackit-sdk-go/services/authorization v0.14.2
 
 require (
 	github.com/golang-jwt/jwt/v5 v5.3.1 // indirect

examples/backgroundrefresh/go.mod

@@ -7,7 +7,7 @@ replace github.com/stackitcloud/stackit-sdk-go/services/dns => ../../services/dn
 
 require (
 	github.com/stackitcloud/stackit-sdk-go/core v0.24.0
-	github.com/stackitcloud/stackit-sdk-go/services/dns v0.17.6
+	github.com/stackitcloud/stackit-sdk-go/services/dns v0.19.2
 )
 
 require (

examples/configuration/go.mod

@@ -7,7 +7,7 @@ replace github.com/stackitcloud/stackit-sdk-go/services/dns => ../../services/dn
 
 require (
 	github.com/stackitcloud/stackit-sdk-go/core v0.24.0
-	github.com/stackitcloud/stackit-sdk-go/services/dns v0.17.6
+	github.com/stackitcloud/stackit-sdk-go/services/dns v0.19.2
 	github.com/stackitcloud/stackit-sdk-go/services/postgresql v0.12.1
 )