diff options
Diffstat (limited to 'internal/route/lfs')
-rw-r--r-- | internal/route/lfs/basic.go | 8 | ||||
-rw-r--r-- | internal/route/lfs/batch.go | 4 | ||||
-rw-r--r-- | internal/route/lfs/route.go | 13 |
3 files changed, 12 insertions, 13 deletions
diff --git a/internal/route/lfs/basic.go b/internal/route/lfs/basic.go index e266c541..f0c2dc8b 100644 --- a/internal/route/lfs/basic.go +++ b/internal/route/lfs/basic.go @@ -12,10 +12,10 @@ import ( "os" "strconv" + "gopkg.in/macaron.v1" log "unknwon.dev/clog/v2" "gogs.io/gogs/internal/conf" - "gogs.io/gogs/internal/context" "gogs.io/gogs/internal/db" "gogs.io/gogs/internal/lfsutil" "gogs.io/gogs/internal/strutil" @@ -28,7 +28,7 @@ const ( ) // GET /{owner}/{repo}.git/info/lfs/object/basic/{oid} -func serveBasicDownload(c *context.Context, repo *db.Repository, oid lfsutil.OID) { +func serveBasicDownload(c *macaron.Context, repo *db.Repository, oid lfsutil.OID) { object, err := db.LFS.GetObjectByOID(repo.ID, oid) if err != nil { if db.IsErrLFSObjectNotExist(err) { @@ -63,7 +63,7 @@ func serveBasicDownload(c *context.Context, repo *db.Repository, oid lfsutil.OID } // PUT /{owner}/{repo}.git/info/lfs/object/basic/{oid} -func serveBasicUpload(c *context.Context, repo *db.Repository, oid lfsutil.OID) { +func serveBasicUpload(c *macaron.Context, repo *db.Repository, oid lfsutil.OID) { // NOTE: LFS client will retry upload the same object if there was a partial failure, // therefore we would like to skip ones that already exist. _, err := db.LFS.GetObjectByOID(repo.ID, oid) @@ -91,7 +91,7 @@ func serveBasicUpload(c *context.Context, repo *db.Repository, oid lfsutil.OID) } // POST /{owner}/{repo}.git/info/lfs/object/basic/verify -func serveBasicVerify(c *context.Context, repo *db.Repository) { +func serveBasicVerify(c *macaron.Context, repo *db.Repository) { var request basicVerifyRequest defer c.Req.Request.Body.Close() err := json.NewDecoder(c.Req.Request.Body).Decode(&request) diff --git a/internal/route/lfs/batch.go b/internal/route/lfs/batch.go index 357aeace..b38435e7 100644 --- a/internal/route/lfs/batch.go +++ b/internal/route/lfs/batch.go @@ -9,17 +9,17 @@ import ( "net/http" jsoniter "github.com/json-iterator/go" + "gopkg.in/macaron.v1" log "unknwon.dev/clog/v2" "gogs.io/gogs/internal/conf" - "gogs.io/gogs/internal/context" "gogs.io/gogs/internal/db" "gogs.io/gogs/internal/lfsutil" "gogs.io/gogs/internal/strutil" ) // POST /{owner}/{repo}.git/info/lfs/object/batch -func serveBatch(c *context.Context, owner *db.User, repo *db.Repository) { +func serveBatch(c *macaron.Context, owner *db.User, repo *db.Repository) { var request batchRequest defer c.Req.Request.Body.Close() err := jsoniter.NewDecoder(c.Req.Request.Body).Decode(&request) diff --git a/internal/route/lfs/route.go b/internal/route/lfs/route.go index b6bd20bf..39ecc348 100644 --- a/internal/route/lfs/route.go +++ b/internal/route/lfs/route.go @@ -13,7 +13,6 @@ import ( log "unknwon.dev/clog/v2" "gogs.io/gogs/internal/authutil" - "gogs.io/gogs/internal/context" "gogs.io/gogs/internal/db" "gogs.io/gogs/internal/lfsutil" ) @@ -44,7 +43,7 @@ func authenticate() macaron.Handler { }) } - return func(c *context.Context) { + return func(c *macaron.Context) { username, password := authutil.DecodeBasic(c.Req.Header) if username == "" { askCredentials(c.Resp) @@ -59,7 +58,7 @@ func authenticate() macaron.Handler { } if err == nil && user.IsEnabledTwoFactor() { - c.PlainText(http.StatusBadRequest, `Users with 2FA enabled are not allowed to authenticate via username and password.`) + c.Error(http.StatusBadRequest, `Users with 2FA enabled are not allowed to authenticate via username and password.`) return } @@ -98,7 +97,7 @@ func authenticate() macaron.Handler { // authorize tries to authorize the user to the context repository with given access mode. func authorize(mode db.AccessMode) macaron.Handler { - return func(c *context.Context, user *db.User) { + return func(c *macaron.Context, user *db.User) { username := c.Params(":username") reponame := strings.TrimSuffix(c.Params(":reponame"), ".git") @@ -137,7 +136,7 @@ func authorize(mode db.AccessMode) macaron.Handler { // verifyHeader checks if the HTTP header contains given value. // When not, response given "failCode" as status code. func verifyHeader(key, value string, failCode int) macaron.Handler { - return func(c *context.Context) { + return func(c *macaron.Context) { if !strings.Contains(c.Req.Header.Get(key), value) { c.Status(failCode) return @@ -147,10 +146,10 @@ func verifyHeader(key, value string, failCode int) macaron.Handler { // verifyOID checks if the ":oid" URL parameter is valid. func verifyOID() macaron.Handler { - return func(c *context.Context) { + return func(c *macaron.Context) { oid := lfsutil.OID(c.Params(":oid")) if !lfsutil.ValidOID(oid) { - c.PlainText(http.StatusBadRequest, "Invalid oid") + c.Error(http.StatusBadRequest, "Invalid oid") return } |