Uploaded image for project: 'Couchbase Gateway'
  1. Couchbase Gateway
  2. CBG-363

Return HTTP 413 when server rejects document due to size

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.6.0
    • SyncGateway
    • Security Level: Public
    • None
    • CBG Sprint 22
    • 3

    Description

      When a document exceeds the server size limit (20MB), SG should return a 413 instead of 500.  This handling was already in place for gocouchbase/gomemcached in ErrorAsHTTPStatus - it needs to get added there for gocb.

      In the case where the xattr exceeds 1MB, it still makes sense to return 500, as that's more of an internal error (i.e. the request payload is not necessarily too big).

       

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          Build sync_gateway-2.6.0-62 contains sync_gateway commit 241dc3f with commit message:
          CBG-363: Add doc too large error (#4120)

          build-team Couchbase Build Team added a comment - Build sync_gateway-2.6.0-62 contains sync_gateway commit 241dc3f with commit message: CBG-363 : Add doc too large error (#4120)

          Build sync_gateway-2.6.0-62 contains sync_gateway commit 241dc3f with commit message:
          CBG-363: Add doc too large error (#4120)

          build-team Couchbase Build Team added a comment - Build sync_gateway-2.6.0-62 contains sync_gateway commit 241dc3f with commit message: CBG-363 : Add doc too large error (#4120)

          Build sync_gateway-2.6.0-62 contains walrus commit 62e1e28 with commit message:
          CBG-363: Add support to give doc too large error (#47)

          build-team Couchbase Build Team added a comment - Build sync_gateway-2.6.0-62 contains walrus commit 62e1e28 with commit message: CBG-363 : Add support to give doc too large error (#47)
          daniel.petersen Daniel Petersen added a comment - QE task filed as:  https://issues.couchbase.com/browse/CM-232

          People

            jacques.rascagneres Jacques Rascagneres
            adamf Adam Fraser
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty