refactor error handling

.vscode/launch.json

@@ -0,0 +1,7 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": []
+}

ap_info.go

@@ -8,6 +8,16 @@ import (
 )
 
 func (api *Api) GetApInfo(siteID model.SiteID, macAddress string) (*model.ApInfo, error) {
+	resp, err := api.getApInfo(siteID, macAddress)
+	if err != nil {
+		return nil, err
+	}
+	return &resp.Result, nil
+}
+
+func (api *Api) getApInfo(siteID model.SiteID, macAddress string) (*model.Response[model.ApInfo], error) {
+	api.refreshMutex.RLock()
+
 	req := ezhttp.Request(
 		ezhttp.Template(api.tmpl),
 		ezhttp.Method("GET"),
@@ -20,14 +30,18 @@ func (api *Api) GetApInfo(siteID model.SiteID, macAddress string) (*model.ApInfo
 
 	resp, err := ezhttp.Do(req)
 	if err != nil {
+		api.refreshMutex.RUnlock()
 		return nil, err
 	}
 	defer resp.Body.Close()
+	api.refreshMutex.RUnlock()
 
-	response, err := ezhttp.ParseJsonResponse[model.Response[model.ApInfo]](resp.Body)
+	response, err := ezhttp.ParseJsonResponse[model.Response[model.ApInfo]](resp)
 	if err != nil {
 		return nil, err
 	}
 
-	return &response.Result, nil
+	return handleResponseErrors(api, response, func() (*model.Response[model.ApInfo], error) {
+		return api.getApInfo(siteID, macAddress)
+	})
 }

api.go

@@ -3,6 +3,8 @@ package omadaapi
 import (
 	"fmt"
 	"net/http"
+	"sync"
+	"time"
 
 	"git.tordarus.net/tordarus/ezhttp"
 	"git.tordarus.net/tordarus/omada-api/model"
@@ -15,6 +17,9 @@ type Api struct {
 
 	accessToken  string
 	refreshToken string
+
+	expiration   time.Time
+	refreshMutex sync.RWMutex
 }
 
 type ApiConfig struct {
@@ -61,6 +66,7 @@ func NewApi(config ApiConfig) (*Api, error) {
 		return nil, fmt.Errorf("auth token request failed: %s", authTokenResponse.Message)
 	}
 
+	api.expiration = time.Now().Add(time.Duration(authTokenResponse.Result.ExpiresIn) * time.Second)
 	api.accessToken = authTokenResponse.Result.AccessToken
 	api.refreshToken = authTokenResponse.Result.RefreshToken
 
@@ -73,6 +79,9 @@ func NewApi(config ApiConfig) (*Api, error) {
 }
 
 func (api *Api) Login() (*model.LoginResponse, error) {
+	api.refreshMutex.RLock()
+	defer api.refreshMutex.RUnlock()
+
 	reqBody := model.LoginRequest{
 		Username: api.config.Username,
 		Password: api.config.Password,
@@ -92,7 +101,7 @@ func (api *Api) Login() (*model.LoginResponse, error) {
 	}
 	defer resp.Body.Close()
 
-	response, err := ezhttp.ParseJsonResponse[model.LoginResponse](resp.Body)
+	response, err := ezhttp.ParseJsonResponse[model.LoginResponse](resp)
 	if err != nil {
 		return nil, err
 	}
@@ -101,6 +110,9 @@ func (api *Api) Login() (*model.LoginResponse, error) {
 }
 
 func (api *Api) AuthCode(csrfToken, sessionID string) (*model.AuthCodeResponse, error) {
+	api.refreshMutex.RLock()
+	defer api.refreshMutex.RUnlock()
+
 	req := ezhttp.Request(
 		ezhttp.Template(api.tmpl),
 		ezhttp.Method("POST"),
@@ -115,7 +127,7 @@ func (api *Api) AuthCode(csrfToken, sessionID string) (*model.AuthCodeResponse,
 	}
 	defer resp.Body.Close()
 
-	response, err := ezhttp.ParseJsonResponse[model.AuthCodeResponse](resp.Body)
+	response, err := ezhttp.ParseJsonResponse[model.AuthCodeResponse](resp)
 	if err != nil {
 		return nil, err
 	}
@@ -124,6 +136,9 @@ func (api *Api) AuthCode(csrfToken, sessionID string) (*model.AuthCodeResponse,
 }
 
 func (api *Api) AuthToken(authCode string) (*model.AuthTokenResponse, error) {
+	api.refreshMutex.RLock()
+	defer api.refreshMutex.RUnlock()
+
 	req := ezhttp.Request(
 		ezhttp.Template(api.tmpl),
 		ezhttp.Method("POST"),
@@ -142,10 +157,44 @@ func (api *Api) AuthToken(authCode string) (*model.AuthTokenResponse, error) {
 	}
 	defer resp.Body.Close()
 
-	response, err := ezhttp.ParseJsonResponse[model.AuthTokenResponse](resp.Body)
+	response, err := ezhttp.ParseJsonResponse[model.AuthTokenResponse](resp)
 	if err != nil {
 		return nil, err
 	}
 
 	return response, nil
 }
+
+func (api *Api) Refresh() error {
+	api.refreshMutex.Lock()
+	defer api.refreshMutex.Unlock()
+
+	req := ezhttp.Request(
+		ezhttp.Template(api.tmpl),
+		ezhttp.Method("POST"),
+		ezhttp.AppendPath("/openapi/authorize/token"),
+		ezhttp.Query(
+			"client_id", api.config.ClientID,
+			"client_secret", api.config.ClientSecret,
+			"refresh_token", api.refreshToken,
+			"grant_type", "refresh_token",
+		),
+	)
+
+	resp, err := ezhttp.Do(req)
+	if err != nil {
+		return err
+	}
+	defer resp.Body.Close()
+
+	response, err := ezhttp.ParseJsonResponse[model.AuthTokenResponse](resp)
+	if err != nil {
+		return err
+	}
+
+	api.expiration = time.Now().Add(time.Duration(response.Result.ExpiresIn) * time.Second)
+	api.accessToken = response.Result.AccessToken
+	api.refreshToken = response.Result.RefreshToken
+
+	return nil
+}

client.go

@@ -4,12 +4,13 @@ import (
 	"fmt"
 	"strconv"
 
+	"git.tordarus.net/tordarus/channel"
 	"git.tordarus.net/tordarus/ezhttp"
 	"git.tordarus.net/tordarus/omada-api/model"
 )
 
-func (api *Api) GetClients(siteID model.SiteID) <-chan *model.Client {
-	out := make(chan *model.Client, 1000)
+func (api *Api) GetClients(siteID model.SiteID) <-chan channel.Result[model.Client] {
+	out := make(chan channel.Result[model.Client], 1000)
 
 	go func() {
 		defer close(out)
@@ -17,11 +18,12 @@ func (api *Api) GetClients(siteID model.SiteID) <-chan *model.Client {
 		for page := 1; ; page++ {
 			resp, err := api.getClients(page, siteID)
 			if err != nil {
+				out <- channel.ResultOf[model.Client](nil, err)
 				return
 			}
 
 			for _, v := range resp.Result.Data {
-				out <- &v
+				out <- channel.ResultOfValue(v, nil)
 			}
 
 			if resp.Result.CurrentPage*resp.Result.CurrentSize >= resp.Result.TotalRows {
@@ -34,6 +36,8 @@ func (api *Api) GetClients(siteID model.SiteID) <-chan *model.Client {
 }
 
 func (api *Api) getClients(page int, siteID model.SiteID) (*model.PagedResponse[model.Client], error) {
+	api.refreshMutex.RLock()
+
 	req := ezhttp.Request(
 		ezhttp.Template(api.tmpl),
 		ezhttp.Method("GET"),
@@ -46,14 +50,18 @@ func (api *Api) getClients(page int, siteID model.SiteID) (*model.PagedResponse[
 
 	resp, err := ezhttp.Do(req)
 	if err != nil {
+		api.refreshMutex.RUnlock()
 		return nil, err
 	}
 	defer resp.Body.Close()
+	api.refreshMutex.RUnlock()
 
-	response, err := ezhttp.ParseJsonResponse[model.PagedResponse[model.Client]](resp.Body)
+	response, err := ezhttp.ParseJsonResponse[model.PagedResponse[model.Client]](resp)
 	if err != nil {
 		return nil, err
 	}
 
-	return response, nil
+	return handlePagedResponseErrors(api, response, func() (*model.PagedResponse[model.Client], error) {
+		return api.getClients(page, siteID)
+	})
 }

device.go

@@ -4,12 +4,13 @@ import (
 	"fmt"
 	"strconv"
 
+	"git.tordarus.net/tordarus/channel"
 	"git.tordarus.net/tordarus/ezhttp"
 	"git.tordarus.net/tordarus/omada-api/model"
 )
 
-func (api *Api) GetDevices(siteID model.SiteID) <-chan *model.Device {
-	out := make(chan *model.Device, 1000)
+func (api *Api) GetDevices(siteID model.SiteID) <-chan channel.Result[model.Device] {
+	out := make(chan channel.Result[model.Device], 1000)
 
 	go func() {
 		defer close(out)
@@ -17,11 +18,12 @@ func (api *Api) GetDevices(siteID model.SiteID) <-chan *model.Device {
 		for page := 1; ; page++ {
 			resp, err := api.getDevices(page, siteID)
 			if err != nil {
+				out <- channel.ResultOf[model.Device](nil, err)
 				return
 			}
 
 			for _, v := range resp.Result.Data {
-				out <- &v
+				out <- channel.ResultOfValue(v, nil)
 			}
 
 			if resp.Result.CurrentPage*resp.Result.CurrentSize >= resp.Result.TotalRows {
@@ -34,6 +36,8 @@ func (api *Api) GetDevices(siteID model.SiteID) <-chan *model.Device {
 }
 
 func (api *Api) getDevices(page int, siteID model.SiteID) (*model.PagedResponse[model.Device], error) {
+	api.refreshMutex.RLock()
+
 	req := ezhttp.Request(
 		ezhttp.Template(api.tmpl),
 		ezhttp.Method("GET"),
@@ -46,14 +50,18 @@ func (api *Api) getDevices(page int, siteID model.SiteID) (*model.PagedResponse[
 
 	resp, err := ezhttp.Do(req)
 	if err != nil {
+		api.refreshMutex.RUnlock()
 		return nil, err
 	}
 	defer resp.Body.Close()
+	api.refreshMutex.RUnlock()
 
-	response, err := ezhttp.ParseJsonResponse[model.PagedResponse[model.Device]](resp.Body)
+	response, err := ezhttp.ParseJsonResponse[model.PagedResponse[model.Device]](resp)
 	if err != nil {
 		return nil, err
 	}
 
-	return response, nil
+	return handlePagedResponseErrors(api, response, func() (*model.PagedResponse[model.Device], error) {
+		return api.getDevices(page, siteID)
+	})
 }

go.mod

@@ -2,4 +2,7 @@ module git.tordarus.net/tordarus/omada-api
 
 go 1.23.0
 
-require git.tordarus.net/tordarus/ezhttp v0.0.3
+require (
+	git.tordarus.net/tordarus/channel v0.1.19
+	git.tordarus.net/tordarus/ezhttp v0.0.5
+)

go.sum

@@ -1,2 +1,4 @@
-git.tordarus.net/tordarus/ezhttp v0.0.3 h1:K6IlLmqkAFUF68HJsOTKcP3ejco7qfm+MuEagohoouo=
-git.tordarus.net/tordarus/ezhttp v0.0.3/go.mod h1:Zq9o0Hibny61GqSCwJHa0PfGjVoUFv/zt2PjiQHXvmY=
+git.tordarus.net/tordarus/channel v0.1.19 h1:d9xnSwFyvBh4B1/82mt0A7Gpm2nIZJTc+9ceJMIOu5Q=
+git.tordarus.net/tordarus/channel v0.1.19/go.mod h1:8/dWFTdGO7g4AeSZ7cF6GerkGbe9c4dBVMVDBxOd9m4=
+git.tordarus.net/tordarus/ezhttp v0.0.5 h1:pxfEdfDeOHT/ATXYy5OQHmeBIho121SBuFvU4ISQ7w0=
+git.tordarus.net/tordarus/ezhttp v0.0.5/go.mod h1:Zq9o0Hibny61GqSCwJHa0PfGjVoUFv/zt2PjiQHXvmY=

site.go

@@ -4,12 +4,13 @@ import (
 	"fmt"
 	"strconv"
 
+	"git.tordarus.net/tordarus/channel"
 	"git.tordarus.net/tordarus/ezhttp"
 	"git.tordarus.net/tordarus/omada-api/model"
 )
 
-func (api *Api) GetSites() <-chan *model.Site {
-	out := make(chan *model.Site, 1000)
+func (api *Api) GetSites() <-chan channel.Result[model.Site] {
+	out := make(chan channel.Result[model.Site], 1000)
 
 	go func() {
 		defer close(out)
@@ -17,11 +18,12 @@ func (api *Api) GetSites() <-chan *model.Site {
 		for page := 1; ; page++ {
 			resp, err := api.getSites(page)
 			if err != nil {
+				out <- channel.ResultOf[model.Site](nil, err)
 				return
 			}
 
 			for _, v := range resp.Result.Data {
-				out <- &v
+				out <- channel.ResultOfValue(v, nil)
 			}
 
 			if resp.Result.CurrentPage*resp.Result.CurrentSize >= resp.Result.TotalRows {
@@ -34,6 +36,8 @@ func (api *Api) GetSites() <-chan *model.Site {
 }
 
 func (api *Api) getSites(page int) (*model.PagedResponse[model.Site], error) {
+	api.refreshMutex.RLock()
+
 	req := ezhttp.Request(
 		ezhttp.Template(api.tmpl),
 		ezhttp.Method("GET"),
@@ -46,14 +50,18 @@ func (api *Api) getSites(page int) (*model.PagedResponse[model.Site], error) {
 
 	resp, err := ezhttp.Do(req)
 	if err != nil {
+		api.refreshMutex.RUnlock()
 		return nil, err
 	}
 	defer resp.Body.Close()
+	api.refreshMutex.RUnlock()
 
-	response, err := ezhttp.ParseJsonResponse[model.PagedResponse[model.Site]](resp.Body)
+	response, err := ezhttp.ParseJsonResponse[model.PagedResponse[model.Site]](resp)
 	if err != nil {
 		return nil, err
 	}
 
-	return response, nil
+	return handlePagedResponseErrors(api, response, func() (*model.PagedResponse[model.Site], error) {
+		return api.getSites(page)
+	})
 }

utils.go

@@ -0,0 +1,49 @@
+package omadaapi
+
+import (
+	"fmt"
+
+	"git.tordarus.net/tordarus/omada-api/model"
+)
+
+const ErrCodeAccessTokenExpired = -44112
+
+func handleResponseErrors[T any](api *Api, response *model.Response[T], retry func() (*model.Response[T], error)) (*model.Response[T], error) {
+	switch response.ErrorCode {
+	case 0:
+		return response, nil
+	case ErrCodeAccessTokenExpired:
+		if err := api.Refresh(); err != nil {
+			return nil, fmt.Errorf("could not refresh access token: %w", err)
+		}
+
+		newResp, err := retry()
+		if err != nil {
+			return nil, err
+		}
+
+		return handleResponseErrors(api, newResp, retry)
+	default:
+		return nil, fmt.Errorf("invalid error code %d with message: %s", response.ErrorCode, response.Message)
+	}
+}
+
+func handlePagedResponseErrors[T any](api *Api, response *model.PagedResponse[T], retry func() (*model.PagedResponse[T], error)) (*model.PagedResponse[T], error) {
+	switch response.ErrorCode {
+	case 0:
+		return response, nil
+	case ErrCodeAccessTokenExpired:
+		if err := api.Refresh(); err != nil {
+			return nil, fmt.Errorf("could not refresh access token: %w", err)
+		}
+
+		newResp, err := retry()
+		if err != nil {
+			return nil, err
+		}
+
+		return handlePagedResponseErrors(api, newResp, retry)
+	default:
+		return nil, fmt.Errorf("invalid error code %d with message: %s", response.ErrorCode, response.Message)
+	}
+}