Skip to content

Commit 9de2144

Browse files
committed
Fix errors in stackit package
1 parent 86bdf1d commit 9de2144

File tree

2 files changed

+15
-45
lines changed

2 files changed

+15
-45
lines changed

pkg/stackit/applicationloadbalancer.go

Lines changed: 10 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,9 @@ package stackit
22

33
import (
44
"context"
5-
"errors"
6-
"net/http"
75

86
"github.com/google/uuid"
97

10-
oapiError "github.com/stackitcloud/stackit-sdk-go/core/oapierror"
118
albsdk "github.com/stackitcloud/stackit-sdk-go/services/alb/v2api"
129
)
1310

@@ -48,7 +45,7 @@ func NewClient(cl *albsdk.APIClient) (Client, error) {
4845
}
4946

5047
func (cl client) GetLoadBalancer(ctx context.Context, projectID, region, name string) (*albsdk.LoadBalancer, error) {
51-
lb, err := cl.client.GetLoadBalancerExecute(ctx, projectID, region, name)
48+
lb, err := cl.client.DefaultAPI.GetLoadBalancer(ctx, projectID, region, name).Execute()
5249
if isOpenAPINotFound(err) {
5350
return lb, ErrorNotFound
5451
}
@@ -57,13 +54,13 @@ func (cl client) GetLoadBalancer(ctx context.Context, projectID, region, name st
5754

5855
// DeleteLoadBalancer returns no error if the load balancer doesn't exist.
5956
func (cl client) DeleteLoadBalancer(ctx context.Context, projectID, region, name string) error {
60-
_, err := cl.client.DeleteLoadBalancerExecute(ctx, projectID, region, name)
57+
_, err := cl.client.DefaultAPI.DeleteLoadBalancer(ctx, projectID, region, name).Execute()
6158
return err
6259
}
6360

6461
// CreateLoadBalancer returns ErrorNotFound if the project is not enabled.
6562
func (cl client) CreateLoadBalancer(ctx context.Context, projectID, region string, create *albsdk.CreateLoadBalancerPayload) (*albsdk.LoadBalancer, error) {
66-
lb, err := cl.client.CreateLoadBalancer(ctx, projectID, region).CreateLoadBalancerPayload(*create).XRequestID(uuid.NewString()).Execute()
63+
lb, err := cl.client.DefaultAPI.CreateLoadBalancer(ctx, projectID, region).CreateLoadBalancerPayload(*create).XRequestID(uuid.NewString()).Execute()
6764
if isOpenAPINotFound(err) {
6865
return lb, ErrorNotFound
6966
}
@@ -73,11 +70,11 @@ func (cl client) CreateLoadBalancer(ctx context.Context, projectID, region strin
7370
func (cl client) UpdateLoadBalancer(ctx context.Context, projectID, region, name string, update *albsdk.UpdateLoadBalancerPayload) (
7471
*albsdk.LoadBalancer, error,
7572
) {
76-
return cl.client.UpdateLoadBalancer(ctx, projectID, region, name).UpdateLoadBalancerPayload(*update).Execute()
73+
return cl.client.DefaultAPI.UpdateLoadBalancer(ctx, projectID, region, name).UpdateLoadBalancerPayload(*update).Execute()
7774
}
7875

7976
func (cl client) UpdateTargetPool(ctx context.Context, projectID, region, name, targetPoolName string, payload albsdk.UpdateTargetPoolPayload) error {
80-
_, err := cl.client.UpdateTargetPool(ctx, projectID, region, name, targetPoolName).UpdateTargetPoolPayload(payload).Execute()
77+
_, err := cl.client.DefaultAPI.UpdateTargetPool(ctx, projectID, region, name, targetPoolName).UpdateTargetPoolPayload(payload).Execute()
8178
return err
8279
}
8380

@@ -87,41 +84,29 @@ func (cl client) CreateCredentials(
8784
region string,
8885
payload albsdk.CreateCredentialsPayload,
8986
) (*albsdk.CreateCredentialsResponse, error) {
90-
return cl.client.CreateCredentials(ctx, projectID, region).CreateCredentialsPayload(payload).XRequestID(uuid.NewString()).Execute()
87+
return cl.client.DefaultAPI.CreateCredentials(ctx, projectID, region).CreateCredentialsPayload(payload).XRequestID(uuid.NewString()).Execute()
9188
}
9289

9390
func (cl client) ListCredentials(ctx context.Context, projectID, region string) (*albsdk.ListCredentialsResponse, error) {
94-
return cl.client.ListCredentialsExecute(ctx, projectID, region)
91+
return cl.client.DefaultAPI.ListCredentials(ctx, projectID, region).Execute()
9592
}
9693

9794
func (cl client) GetCredentials(ctx context.Context, projectID, region, credentialsRef string) (*albsdk.GetCredentialsResponse, error) {
98-
return cl.client.GetCredentialsExecute(ctx, projectID, region, credentialsRef)
95+
return cl.client.DefaultAPI.GetCredentials(ctx, projectID, region, credentialsRef).Execute()
9996
}
10097

10198
func (cl client) UpdateCredentials(ctx context.Context, projectID, region, credentialsRef string, payload albsdk.UpdateCredentialsPayload) error {
102-
_, err := cl.client.UpdateCredentials(ctx, projectID, region, credentialsRef).UpdateCredentialsPayload(payload).Execute()
99+
_, err := cl.client.DefaultAPI.UpdateCredentials(ctx, projectID, region, credentialsRef).UpdateCredentialsPayload(payload).Execute()
103100
if err != nil {
104101
return err
105102
}
106103
return nil
107104
}
108105

109106
func (cl client) DeleteCredentials(ctx context.Context, projectID, region, credentialsRef string) error {
110-
_, err := cl.client.DeleteCredentials(ctx, projectID, region, credentialsRef).Execute()
107+
_, err := cl.client.DefaultAPI.DeleteCredentials(ctx, projectID, region, credentialsRef).Execute()
111108
if err != nil {
112109
return err
113110
}
114111
return nil
115112
}
116-
117-
func isOpenAPINotFound(err error) bool {
118-
apiErr := &oapiError.GenericOpenAPIError{}
119-
if !errors.As(err, &apiErr) {
120-
return false
121-
}
122-
return apiErr.StatusCode == http.StatusNotFound
123-
}
124-
125-
func IsNotFound(err error) bool {
126-
return errors.Is(err, ErrorNotFound)
127-
}

pkg/stackit/applicationloadbalancercertificates.go

Lines changed: 5 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,7 @@ package stackit
22

33
import (
44
"context"
5-
"errors"
6-
"net/http"
75

8-
oapiError "github.com/stackitcloud/stackit-sdk-go/core/oapierror"
96
certsdk "github.com/stackitcloud/stackit-sdk-go/services/certificates/v2api"
107
)
118

@@ -22,44 +19,32 @@ type certClient struct {
2219

2320
var _ CertificatesClient = (*certClient)(nil)
2421

25-
func NewCertClient(cl *certsdk.APIClient) (Client, error) {
22+
func NewCertClient(cl *certsdk.APIClient) (CertificatesClient, error) {
2623
return &certClient{client: cl}, nil
2724
}
2825

2926
func (cl certClient) GetCertificate(ctx context.Context, projectID, region, name string) (*certsdk.GetCertificateResponse, error) {
30-
cert, err := cl.client.GetCertificateExecute(ctx, projectID, region, name)
27+
cert, err := cl.client.DefaultAPI.GetCertificate(ctx, projectID, region, name).Execute()
3128
if isOpenAPINotFound(err) {
3229
return cert, ErrorNotFound
3330
}
3431
return cert, err
3532
}
3633

3734
func (cl certClient) DeleteCertificate(ctx context.Context, projectID, region, name string) error {
38-
_, err := cl.client.DeleteCertificateExecute(ctx, projectID, region, name)
35+
_, err := cl.client.DefaultAPI.DeleteCertificate(ctx, projectID, region, name).Execute()
3936
return err
4037
}
4138

4239
func (cl certClient) CreateCertificate(ctx context.Context, projectID, region string, certificate *certsdk.CreateCertificatePayload) (*certsdk.GetCertificateResponse, error) {
43-
cert, err := cl.client.CreateCertificate(ctx, projectID, region).CreateCertificatePayload(*certificate).Execute()
40+
cert, err := cl.client.DefaultAPI.CreateCertificate(ctx, projectID, region).CreateCertificatePayload(*certificate).Execute()
4441
if isOpenAPINotFound(err) {
4542
return cert, ErrorNotFound
4643
}
4744
return cert, err
4845
}
4946

5047
func (cl certClient) ListCertificate(ctx context.Context, projectID, region string) (*certsdk.ListCertificatesResponse, error) {
51-
certs, err := cl.client.ListCertificates(ctx, projectID, region).Execute()
48+
certs, err := cl.client.DefaultAPI.ListCertificates(ctx, projectID, region).Execute()
5249
return certs, err
5350
}
54-
55-
func isOpenAPINotFound(err error) bool {
56-
apiErr := &oapiError.GenericOpenAPIError{}
57-
if !errors.As(err, &apiErr) {
58-
return false
59-
}
60-
return apiErr.StatusCode == http.StatusNotFound
61-
}
62-
63-
func IsNotFound(err error) bool {
64-
return errors.Is(err, ErrorNotFound)
65-
}

0 commit comments

Comments
 (0)