Mailing-List: contact dev-help@trafficserver.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@trafficserver.apache.org
From: ericcarlschwartz <git@git.apache.org>
To: dev@trafficserver.apache.org
Reply-To: dev@trafficserver.apache.org
Message-ID: <git-pr-157-trafficserver@git.apache.org>
Subject: [GitHub] trafficserver pull request: [TS-2157] Replace "addr" with
 appropri...
Content-Type: text/plain
Date: Tue, 23 Dec 2014 21:52:07 +0000 (UTC)

GitHub user ericcarlschwartz opened a pull request:

    https://github.com/apache/trafficserver/pull/157

    [TS-2157] Replace "addr" with appropriate "src_addr" and "dst_addr" in ConnectionAttributes

    Have made initial changes.  No added functionality as of yet.  Have tested and performs the same as previous addr/port version in my tests.  Wondering if someone could give this a look/if we should extend api to allow for more control over local_addr and remote_addr.  There will also be some occasions where local_addr remains unused do we want to set it in all cases?  If so, where? Feel free to let me know in code review.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/yahoo/trafficserver TS-2157

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafficserver/pull/157.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #157
    
----
commit 0cb7abe01e445552cee8d16b2f16e16925ac92f8
Author: es <es@yahoo-inc.com>
Date:   2014-12-16T23:33:27Z

    changes to ConnectionAttributes struct

commit dc155ffbcd00be9db8fe5525d63f11d6f43a2866
Author: es <es@yahoo-inc.com>
Date:   2014-12-22T20:06:18Z

    rename members

commit 852c7ecf0e3a8af194bef5d032b0b3c13bd6758d
Author: es <es@yahoo-inc.com>
Date:   2014-12-23T18:17:39Z

    Changes for TS-2157

commit e4885da878b47cd463c065cf75b6564811553c91
Author: es <es@yahoo-inc.com>
Date:   2014-12-23T18:37:07Z

    fix non client ports

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---