Details

    • Type: Bug
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 1.1.0-dp4
    • Fix Version/s: 1.1.0-dp5
    • Component/s: library
    • Security Level: Public
    • Labels:
      None

      Description

      Negative expiry values should throw an error. It is impossible to determine if the user means to disable expiry, or has accidentally make the expiry value too low (which in C-land will cause a wrap-around).

      Currently it just wraps around to a time_t (or doesn't throw an error at least).

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

        Activity

        Hide
        mjackson Michael Jackson added a comment -

        1.0.x branch fix and tests for same: http://review.couchbase.org/#/c/21079/

        Show
        mjackson Michael Jackson added a comment - 1.0.x branch fix and tests for same: http://review.couchbase.org/#/c/21079/
        Hide
        mjackson Michael Jackson added a comment -

        1.1.x/ master fix for same: http://review.couchbase.org/#/c/21090/

        (These may need refactored tests, those will come in a separate issue.)

        Show
        mjackson Michael Jackson added a comment - 1.1.x/ master fix for same: http://review.couchbase.org/#/c/21090/ (These may need refactored tests, those will come in a separate issue.)

          People

          • Assignee:
            mjackson Michael Jackson
            Reporter:
            mnunberg Mark Nunberg
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes