Skip to content

Provide original req to Service#769

Merged
jsdevel merged 1 commit into
vpulim:masterfrom
evantahler:master
Oct 31, 2015
Merged

Provide original req to Service#769
jsdevel merged 1 commit into
vpulim:masterfrom
evantahler:master

Conversation

@evantahler

Copy link
Copy Markdown
Contributor

Thanks for this great project!

Sometimes you may want the original req passed to your service methods. In the simplest case, you might just want to log the requestor's IP address.

This PR passes req to the service methods as a 4th optional param.

@jsdevel

jsdevel commented Oct 30, 2015

Copy link
Copy Markdown
Collaborator

@evantahler can you squash down to one commit?

note new option in readme

test for req within Service

spacing

update test for ipv6
@evantahler

Copy link
Copy Markdown
Contributor Author

sure thing. Squashed.

jsdevel added a commit that referenced this pull request Oct 31, 2015
Provide original `req` to Service
@jsdevel
jsdevel merged commit b4ac994 into vpulim:master Oct 31, 2015
@jsdevel

jsdevel commented Oct 31, 2015

Copy link
Copy Markdown
Collaborator

Thanks @evantahler!

@evantahler

Copy link
Copy Markdown
Contributor Author

Cool. Any plans on a new release to NPM?

@jsdevel

jsdevel commented Oct 31, 2015

Copy link
Copy Markdown
Collaborator

v0.11.0

diarmaidm pushed a commit to diarmaidm/node-soap that referenced this pull request Feb 3, 2016
Provide original `req` to Service
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants