-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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 possibility to add action to content-type header #1073
added possibility to add action to content-type header #1073
Conversation
if (!this.wsdl.options.forceSoap12Headers) { | ||
if (this.wsdl.options.forceSoap12Headers) { | ||
headers['Content-Type'] = 'application/soap+xml; charset=utf-8'; | ||
if (this.wsdl.options.addHeadersAction) { |
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.
According to specification from https://docs.intersystems.com/latest/csp/docbook/DocBook.UI.Page.cls?KEY=ROBJ_method_soapaction action param should be added within forceSoap12Headers flag, not some additional one as because it is required in 1.2 version
thanks! |
@@ -94,6 +94,8 @@ export interface IWsdlBaseOptions { | |||
wsdl_options?: { [key: string]: any }; | |||
/** set proper headers for SOAP v1.2. */ | |||
forceSoap12Headers?: boolean; | |||
/** set content type header action for SOAP v1.2 */ | |||
addHeadersAction?: boolean; |
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.
@gbar can you follow up with a PR to add this to the README please?
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.
Will add that when I get home. Will create another PR with that
I have added support to add soapAction to Content-Type header.
For example:
https://docs.intersystems.com/latest/csp/docbook/DocBook.UI.Page.cls?KEY=ROBJ_method_soapaction
It allows to set Content-Type to
'application/soap+xml; charset=utf-8'; action=Action
Added option param:
addHeadersAction
that allows to turn this on/off.