Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

added forceMTOM option and updated the Readme #1086

Merged
merged 5 commits into from
Aug 8, 2019
Merged

added forceMTOM option and updated the Readme #1086

merged 5 commits into from
Aug 8, 2019

Conversation

g-m-a
Copy link
Contributor

@g-m-a g-m-a commented Aug 8, 2019 •
edited
Loading

  • cleaned some of the previous code (lame console.log leftover)
  • added forceMTOM in the request options so that you can send MTOM request even without having binary attachments to the request. Magically enough some services always use MTOM.
  • updated the Readme with some info on the attachments and forceMTOM options

@coveralls
Copy link

Coverage Status

Coverage remained the same at 94.476% when pulling bc23539 on g-m-a:master into 5af5fab on vpulim:master.

@jsdevel jsdevel merged commit aee79a8 into vpulim:master Aug 8, 2019
@jsdevel
Copy link
Collaborator

jsdevel commented Aug 8, 2019

thanks!

@snyk-bot snyk-bot mentioned this pull request Dec 3, 2019
Closed
Danail-Irinkov pushed a commit to Danail-Irinkov/node-soap that referenced this pull request Jan 19, 2020
@g-m-a @Danail-Irinkov
added forceMTOM option and updated the Readme (vpulim#1086)
9f17e8e 
* fixed MTOM removing soap12header

* cleaned code + added test

* added forceMTOM extra request option to send MTOM requests even if no attachement

* Update Readme.md
This was referenced Mar 15, 2020
Merged
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@g-m-a @coveralls @jsdevel