- Notifications
You must be signed in to change notification settings - Fork425
Setting source port#551
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Setting source port#551
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Please rename this assource_port instead to follow the convention already set in the class.
deanberris commentedAug 19, 2015
Just a couple of comments, sorry for the very long delay@DanielBujnik -- I'm usually better about these things. 😏 |
DanielBujnik commentedAug 19, 2015
Thanks for feedback. Getter and setter methods are now named as source_port as requested. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Please use boost::uint16_t instead ofunsigned short.
deanberris commentedAug 20, 2015
Sorry about the back-and forth here, just a couple more things. |
DanielBujnik commentedAug 20, 2015
OK done. |
deanberris commentedAug 25, 2015
LGTM -- thanks@DanielBujnik, merging now. |
With this change it is possible to set the TCP source port of HTTP and HTTPS client requests by calling setSourcePort method of client::request object.