Uploaded image for project: 'Traffic Server'
  1. Traffic Server
  2. TS-535

INKHttpTxnSetHttpRetStatus not honored when an API transaction is reenabled with INK_EVENT_HTTP_ERROR

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.1.3
    • 2.1.5
    • TS API
    • None

    Description

      imagine a plugin inside READ_REQUEST like:

      ....
      INKHttpTxnSetHttpRetStatus(txnp, INK_HTTP_STATUS_NOT_FOUND);
      ......
      INKHttpTxnReenable(txnp, INK_EVENT_HTTP_ERROR);

      will always returns HTTP/1.1 500 INKApi Error

      I think it should honor errors >=400 a plugin writer has set. This is important inside READ_REQUEST where I may want to bail the rest of the state machine and return something (other than 500) to the client immediately.

      Attachments

        1. patch.txt
          1 kB
          Eric Balsa

        Activity

          People

            ericb Eric Balsa
            ericb Eric Balsa
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 1h
                1h
                Remaining:
                Remaining Estimate - 1h
                1h
                Logged:
                Time Spent - Not Specified
                Not Specified