feat: optimize code

provider.go

@@ -5,8 +5,9 @@ import (
 	"slices"
 	"time"
 
-	"github.com/libdns/huaweicloud/sdk/services/dns/v2/model"
 	"github.com/libdns/libdns"
+
+	"github.com/libdns/huaweicloud/sdk/services/dns/v2/model"
 )
 
 // Provider facilitates DNS record manipulation with Huawei Cloud

sdk/core/auth/basic/basic_icredential.go

@@ -21,17 +21,18 @@ package basic
 
 import (
 	"fmt"
+	"io/ioutil"
+	"os"
+	"strings"
+	"time"
+
 	"github.com/libdns/huaweicloud/sdk/core/auth"
 	"github.com/libdns/huaweicloud/sdk/core/auth/cache"
 	"github.com/libdns/huaweicloud/sdk/core/auth/internal"
 	"github.com/libdns/huaweicloud/sdk/core/auth/signer"
 	"github.com/libdns/huaweicloud/sdk/core/impl"
 	"github.com/libdns/huaweicloud/sdk/core/request"
 	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
-	"io/ioutil"
-	"os"
-	"strings"
-	"time"
 )
 
 const (
@@ -320,15 +321,6 @@ func (builder *CredentialsBuilder) WithIdTokenFile(idTokenFile string) *Credenti
 	return builder
 }
 
-// Deprecated: This function may panic under certain circumstances. Use SafeBuild instead.
-func (builder *CredentialsBuilder) Build() *Credentials {
-	credentials, err := builder.SafeBuild()
-	if err != nil {
-		panic(err)
-	}
-	return credentials
-}
-
 func (builder *CredentialsBuilder) SafeBuild() (*Credentials, error) {
 	if builder.errMap != nil && len(builder.errMap) != 0 {
 		errMsg := "build credentials failed: "

sdk/core/auth/derived_icredential.go

@@ -20,9 +20,10 @@
 package auth
 
 import (
-	"github.com/libdns/huaweicloud/sdk/core/request"
 	"regexp"
 	"strings"
+
+	"github.com/libdns/huaweicloud/sdk/core/request"
 )
 
 const DefaultEndpointReg = "^[a-z][a-z0-9-]+(\\.[a-z]{2,}-[a-z]+-\\d{1,2})?\\.(my)?(huaweicloud|myhwclouds).(com|cn)"

sdk/core/auth/global/global_icredential.go

@@ -21,16 +21,17 @@ package global
 
 import (
 	"fmt"
+	"io/ioutil"
+	"os"
+	"time"
+
 	"github.com/libdns/huaweicloud/sdk/core/auth"
 	"github.com/libdns/huaweicloud/sdk/core/auth/cache"
 	"github.com/libdns/huaweicloud/sdk/core/auth/internal"
 	"github.com/libdns/huaweicloud/sdk/core/auth/signer"
 	"github.com/libdns/huaweicloud/sdk/core/impl"
 	"github.com/libdns/huaweicloud/sdk/core/request"
 	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
-	"io/ioutil"
-	"os"
-	"time"
 )
 
 const (
@@ -311,15 +312,6 @@ func (builder *CredentialsBuilder) WithIdTokenFile(idTokenFile string) *Credenti
 	return builder
 }
 
-// Deprecated: This function may panic under certain circumstances. Use SafeBuild instead.
-func (builder *CredentialsBuilder) Build() *Credentials {
-	credentials, err := builder.SafeBuild()
-	if err != nil {
-		panic(err)
-	}
-	return credentials
-}
-
 func (builder *CredentialsBuilder) SafeBuild() (*Credentials, error) {
 	if builder.errMap != nil && len(builder.errMap) != 0 {
 		errMsg := "build credentials failed: "

sdk/core/auth/internal/iam_service.go

@@ -21,15 +21,16 @@ package internal
 
 import (
 	"errors"
+	"os"
+	"reflect"
+	"strings"
+
 	"github.com/libdns/huaweicloud/sdk/core/config"
 	"github.com/libdns/huaweicloud/sdk/core/impl"
 	"github.com/libdns/huaweicloud/sdk/core/request"
 	"github.com/libdns/huaweicloud/sdk/core/response"
 	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
 	"github.com/libdns/huaweicloud/sdk/core/utils"
-	"os"
-	"reflect"
-	"strings"
 )
 
 const (

sdk/core/auth/internal/metadata.go

@@ -20,11 +20,12 @@
 package internal
 
 import (
-	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
-	"github.com/libdns/huaweicloud/sdk/core/utils"
 	"io/ioutil"
 	"net/http"
 	"time"
+
+	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
+	"github.com/libdns/huaweicloud/sdk/core/utils"
 )
 
 type GetTemporaryCredentialFromMetadataResponse struct {

sdk/core/auth/provider/credential_provider.go

@@ -21,9 +21,10 @@ package provider
 
 import (
 	"fmt"
+	"reflect"
+
 	"github.com/libdns/huaweicloud/sdk/core/auth"
 	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
-	"reflect"
 )
 
 const (

sdk/core/auth/provider/env_provider.go

@@ -20,12 +20,13 @@
 package provider
 
 import (
+	"os"
+	"strings"
+
 	"github.com/libdns/huaweicloud/sdk/core/auth"
 	"github.com/libdns/huaweicloud/sdk/core/auth/basic"
 	"github.com/libdns/huaweicloud/sdk/core/auth/global"
 	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
-	"os"
-	"strings"
 )
 
 const (
@@ -70,14 +71,14 @@ func (p *EnvCredentialProvider) GetCredentials() (auth.ICredential, error) {
 		if err != nil {
 			return nil, err
 		}
-		return builder.Build(), nil
+		return builder.SafeBuild()
 	} else if strings.HasPrefix(p.credentialType, globalCredentialType) {
 		builder := global.NewCredentialsBuilder().WithDomainId(os.Getenv(domainIdEnvName))
 		err := fillCommonAttrs(builder, getCommonAttrsFromEnv())
 		if err != nil {
 			return nil, err
 		}
-		return builder.Build(), nil
+		return builder.SafeBuild()
 	}
 
 	return nil, sdkerr.NewCredentialsTypeError("unsupported credential type: " + p.credentialType)

sdk/core/auth/provider/metadata_provider.go

@@ -20,11 +20,12 @@
 package provider
 
 import (
+	"strings"
+
 	"github.com/libdns/huaweicloud/sdk/core/auth"
 	"github.com/libdns/huaweicloud/sdk/core/auth/basic"
 	"github.com/libdns/huaweicloud/sdk/core/auth/global"
 	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
-	"strings"
 )
 
 type MetadataCredentialProvider struct {
@@ -54,18 +55,22 @@ func (p *MetadataCredentialProvider) GetCredentials() (auth.ICredential, error)
 	}
 
 	if strings.HasPrefix(p.credentialType, basicCredentialType) {
-		credentials := basic.NewCredentialsBuilder().Build()
-		err := credentials.UpdateSecurityTokenFromMetadata()
+		credentials, err := basic.NewCredentialsBuilder().SafeBuild()
 		if err != nil {
 			return nil, err
 		}
+		if err = credentials.UpdateSecurityTokenFromMetadata(); err != nil {
+			return nil, err
+		}
 		return credentials, nil
 	} else if strings.HasPrefix(p.credentialType, globalCredentialType) {
-		credentials := global.NewCredentialsBuilder().Build()
-		err := credentials.UpdateSecurityTokenFromMetadata()
+		credentials, err := global.NewCredentialsBuilder().SafeBuild()
 		if err != nil {
 			return nil, err
 		}
+		if err = credentials.UpdateSecurityTokenFromMetadata(); err != nil {
+			return nil, err
+		}
 		return credentials, nil
 	}
 

sdk/core/auth/provider/profile_provider.go

@@ -21,8 +21,9 @@ package provider
 
 import (
 	"errors"
-	"github.com/libdns/huaweicloud/sdk/core/auth"
 	"strings"
+
+	"github.com/libdns/huaweicloud/sdk/core/auth"
 )
 
 const (

sdk/core/auth/provider/provider_chain.go

@@ -20,9 +20,10 @@
 package provider
 
 import (
+	"strings"
+
 	"github.com/libdns/huaweicloud/sdk/core/auth"
 	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
-	"strings"
 )
 
 type CredentialProviderChain struct {

sdk/core/auth/signer/derived_signer.go

@@ -24,11 +24,13 @@ import (
 	"encoding/hex"
 	"errors"
 	"fmt"
-	"github.com/libdns/huaweicloud/sdk/core/request"
-	"golang.org/x/crypto/hkdf"
 	"io"
 	"strings"
 	"time"
+
+	"golang.org/x/crypto/hkdf"
+
+	"github.com/libdns/huaweicloud/sdk/core/request"
 )
 
 const (

sdk/core/auth/signer/iaksksigner.go

@@ -21,6 +21,7 @@ package signer
 
 import (
 	"errors"
+
 	"github.com/libdns/huaweicloud/sdk/core/auth/signer/algorithm"
 	"github.com/libdns/huaweicloud/sdk/core/request"
 )

sdk/core/auth/signer/p256_sha256_signer.go

@@ -25,8 +25,9 @@ import (
 	"crypto/elliptic"
 	"encoding/hex"
 	"errors"
-	"github.com/libdns/huaweicloud/sdk/core/request"
 	"math/big"
+
+	"github.com/libdns/huaweicloud/sdk/core/request"
 )
 
 const (

sdk/core/auth/signer/signer.go

@@ -9,12 +9,13 @@ import (
 	"encoding/hex"
 	"errors"
 	"fmt"
-	"github.com/libdns/huaweicloud/sdk/core/request"
 	"net/url"
 	"reflect"
 	"sort"
 	"strings"
 	"time"
+
+	"github.com/libdns/huaweicloud/sdk/core/request"
 )
 
 const (

sdk/core/config/http_config.go

@@ -22,11 +22,12 @@ package config
 import (
 	"context"
 	"fmt"
-	"github.com/libdns/huaweicloud/sdk/core/auth/signer/algorithm"
-	"github.com/libdns/huaweicloud/sdk/core/httphandler"
 	"net"
 	"net/http"
 	"time"
+
+	"github.com/libdns/huaweicloud/sdk/core/auth/signer/algorithm"
+	"github.com/libdns/huaweicloud/sdk/core/httphandler"
 )
 
 const (

sdk/core/def/multipart.go

@@ -22,13 +22,14 @@ package def
 import (
 	"errors"
 	"fmt"
-	"github.com/libdns/huaweicloud/sdk/core/converter"
 	"io"
 	"mime/multipart"
 	"net/textproto"
 	"os"
 	"path/filepath"
 	"strings"
+
+	"github.com/libdns/huaweicloud/sdk/core/converter"
 )
 
 var quoteEscape = strings.NewReplacer("\\", "\\\\", `"`, "\\\"")

sdk/core/hc_http_client.go

@@ -24,6 +24,12 @@ import (
 	"encoding/xml"
 	"errors"
 	"fmt"
+	"net"
+	"net/url"
+	"reflect"
+	"strings"
+	"sync/atomic"
+
 	"github.com/libdns/huaweicloud/sdk/core/auth"
 	"github.com/libdns/huaweicloud/sdk/core/converter"
 	"github.com/libdns/huaweicloud/sdk/core/def"
@@ -34,11 +40,6 @@ import (
 	"github.com/libdns/huaweicloud/sdk/core/response"
 	"github.com/libdns/huaweicloud/sdk/core/sdkerr"
 	"github.com/libdns/huaweicloud/sdk/core/utils"
-	"net"
-	"net/url"
-	"reflect"
-	"strings"
-	"sync/atomic"
 )
 
 const (
@@ -85,11 +86,6 @@ func (hc *HcHttpClient) GetCredential() auth.ICredential {
 	return hc.credential
 }
 
-// Deprecated: This function will be removed in the future version. Use WithExtraHeaders instead.
-func (hc *HcHttpClient) PreInvoke(headers map[string]string) *HcHttpClient {
-	return hc.WithExtraHeaders(headers)
-}
-
 func (hc *HcHttpClient) Sync(req interface{}, reqDef *def.HttpRequestDef) (interface{}, error) {
 	exg := &exchange.SdkExchange{
 		ApiReference: &exchange.ApiReference{},