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

Include protocol in readme examples setting endpoint from wsdl_domain #477

Merged
merged 1 commit into from
May 25, 2021

Conversation

cgunther
Copy link
Contributor

@cgunther cgunther commented May 25, 2021

The wsdl_domain is just a domain/hostname, no protocol, however endpoint expects to include a protocol, so when you're buliding the endpoint from the wsdl_domain, you need to prefix it with a protocol.

#473 introduced the ability to set the endpoint.

The `wsdl_domain` is just a domain/hostname, no protocol, however
`endpoint` expects to include a protocol, so when you're buliding the
`endpoint` from the `wsdl_domain`, you need to prefix it with a
protocol.
@iloveitaly
Copy link
Member

Thanks!

@iloveitaly iloveitaly merged commit 33594f4 into NetSweet:master May 25, 2021
@cgunther cgunther deleted the endpoint-protocol branch May 25, 2021 21:32
diegopolido pushed a commit to penrosehill/netsuite that referenced this pull request Oct 7, 2021
…NetSweet#477)

The `wsdl_domain` is just a domain/hostname, no protocol, however
`endpoint` expects to include a protocol, so when you're buliding the
`endpoint` from the `wsdl_domain`, you need to prefix it with a
protocol.
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