larryh/corso
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

GOFMT: Added defer clause for writer.Close().

Browse Source
This commit is contained in:
Danny Adams 2023-01-06 16:57:25 -05:00
parent d2b7ac0d0c
commit 5ac97ec856
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/internal/connector/graph/service.go
View File

@ -7,6 +7,7 @@ import (
msgraphsdk "github.com/microsoftgraph/msgraph-sdk-go"
"github.com/pkg/errors"
"github.com/alcionai/corso/src/internal/connector/support"
"github.com/alcionai/corso/src/pkg/account"
"github.com/alcionai/corso/src/pkg/path"
)
@ -47,7 +48,12 @@ func (s Service) Client() *msgraphsdk.GraphServiceClient {
func (s Service) Serialize(object absser.Parsable) ([]byte, error) {
writer, err := s.adapter.GetSerializationWriterFactory().GetSerializationWriter("application/json")
defer writer.Close()
defer func() {
writerErr := writer.Close()
if writerErr != nil {
err = support.WrapAndAppend("failure during writer closer", writerErr, err)
}
}()
if err != nil || writer == nil {
return nil, errors.Wrap(err, "creating json serialization writer")