Uploaded image for project: 'CouchDB'
  1. CouchDB
  2. COUCHDB-1421

Wrong X-Forwarded-For address chosen as "peer"

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Won't Fix
    • None
    • None
    • HTTP Interface
    • None

    Description

      I noticed that in the Mochiweb code, it uses the last item of the X-Forwarded-For list as the peer:
      https://github.com/apache/couchdb/blob/master/src/mochiweb/mochiweb_request.erl#L82

      But shouldn't this snag the first item of the list instead? http://tools.ietf.org/html/draft-petersson-forwarded-for-02#section-5.2 says "the first for-parameter will disclose the user agent where the request first was made" — the user agent is what I'd want as an app developer, not the second-nearest proxy.

      Attachments

        Activity

          People

            Unassigned Unassigned
            natevw Nathan Vander Wilt
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: