Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support to add/customize request options (Ex: timeout, proxy etc.) #32

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

maulik07
Copy link

@maulik07 maulik07 commented Sep 20, 2016

In response to Issue #31
Addresses PR #8

Added support to customize and tweak all available options listed under request module for node: Available Options.

The options could be passed in as a JSON Object to constructor argument of Http handler which will in-turn club these options with the required/default ones while sending a request.

The constructor argument could be omitted/skipped, under which case the API sticks to its current core functionality.

Sample Usage:
var handlers = [ new Security({}, [new UsernameToken({username: "yaron", password: "1234"})]) , new Http({timeout: 3000}) ]

Maulik Hakani added 2 commits September 20, 2016 11:32
…while sending ws requests. Options could be passed as constructor args to http handler.
@maulik07 maulik07 changed the title Added support to timeout option Added support to add/customize request options (Ex: timeout, proxy etc.) Sep 20, 2016
@Jonlondon
Copy link

Could be strange but in my case I need to edit headers. Should it be more interesting to replace 'jsonConcat' by an extend functionality so that we can replace any options?

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