Skip to content

Commit 748dc78

Browse files
committed
Fix nakedreturns after merge
1 parent 650d02a commit 748dc78

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

modules/activitypub/client.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ func NewClient(user *user_model.User, pubID string) (c *Client, err error) {
9292
priv: privParsed,
9393
pubID: pubID,
9494
}
95-
return
95+
return c, err
9696
}
9797

9898
// NewRequest function
@@ -110,7 +110,7 @@ func (c *Client) NewRequest(b []byte, to string) (req *http.Request, err error)
110110
return
111111
}
112112
err = signer.SignRequest(c.priv, c.pubID, req, b)
113-
return
113+
return req, err
114114
}
115115

116116
// Post function
@@ -120,5 +120,5 @@ func (c *Client) Post(b []byte, to string) (resp *http.Response, err error) {
120120
return
121121
}
122122
resp, err = c.client.Do(req)
123-
return
123+
return resp, err
124124
}

modules/activitypub/user_settings.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,11 @@ func GetKeyPair(user *user_model.User) (pub, priv string, err error) {
3535
// GetPublicKey function returns a user's public key
3636
func GetPublicKey(user *user_model.User) (pub string, err error) {
3737
pub, _, err = GetKeyPair(user)
38-
return
38+
return pub, err
3939
}
4040

4141
// GetPrivateKey function returns a user's private key
4242
func GetPrivateKey(user *user_model.User) (priv string, err error) {
4343
_, priv, err = GetKeyPair(user)
44-
return
44+
return priv, err
4545
}

routers/api/v1/activitypub/reqsignature.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ func getPublicKeyFromResponse(b []byte, keyID *url.URL) (p crypto.PublicKey, err
4141
return
4242
}
4343
p, err = x509.ParsePKIXPublicKey(block.Bytes)
44-
return
44+
return p, err
4545
}
4646

4747
func fetch(iri *url.URL) (b []byte, err error) {
@@ -59,7 +59,7 @@ func fetch(iri *url.URL) (b []byte, err error) {
5959
return
6060
}
6161
b, err = io.ReadAll(io.LimitReader(resp.Body, setting.Federation.MaxSize))
62-
return
62+
return b, err
6363
}
6464

6565
func verifyHTTPSignatures(ctx *gitea_context.APIContext) (authenticated bool, err error) {
@@ -87,7 +87,7 @@ func verifyHTTPSignatures(ctx *gitea_context.APIContext) (authenticated bool, er
8787
// 3. Verify the other actor's key
8888
algo := httpsig.Algorithm(setting.Federation.Algorithms[0])
8989
authenticated = v.Verify(pubKey, algo) == nil
90-
return
90+
return authenticated, err
9191
}
9292

9393
// ReqHTTPSignature function

0 commit comments

Comments
 (0)