hooklift / gowsdl

WSDL2Go code generation as well as its SOAP proxy
Mozilla Public License 2.0
1.15k stars 389 forks source link

NRCS Envelope Fault #17

Open EntilZha opened 9 years ago

EntilZha commented 9 years ago

Got around to working more on the NRCS wsdl service I was interested. Ran into another problem, like before open to helping, but some insight/maybe fix would be great. I am using the code from #15

Below is what I wrote additionally to query all stations which are also snotels, so using networkCd of SNTL.

func GetSnotelStations() ([]string, error) {
    request := &getStations{NetworkCds: []string{"SNTL"}, LogicalAnd: true}
    stations, err := service.GetStations(request)
    if err != nil {
        return nil, err
    }
    return stations.Return_, err
}

When I do this, I get the following error: PANIC: expected element type <getStationsResponse> but have <Fault>

Wireshark shows these request contents, couldn't find a way to copy/paste so they are images out 500a 500b

Here is the wireshark of my pre-existing ruby code that makes a successful request. correct

Thanks and let me know how else I can help with info/etc

c4milo commented 9 years ago

It seems to be a namespace issue. Go XML package doesn't support namespace prefixes yet but it does set the namespace attribute on each element explicitly instead. That NRCS service doesn't seem to be aware of this and is rejecting what I would consider a valid SOAP request that is at the same time WS-I compliant.

Assuming they are using Java, by default the native Java DOMBuilderFactory is configured to ignore namespaces, thus <getStations xmlns="http://www.wcc.nrcs.usda.gov/ns/awdbWebService"></getStations> is considered an unexpected element. They need to instruct their unmarshaller to be aware of namespaces in elements, like so: http://stackoverflow.com/a/24399323/2807845. But, considering that's a government service, I guess, we will have to work around that issue somehow. Any help is very much appreciated as I'm out of extra cycles right now.

EntilZha commented 9 years ago

If you could give some direction, I could try working something out. I also actually know some of the people that work for the NRCS on this service in particular, although unlikely, might be able to do something on that end.

c4milo commented 9 years ago

FWIW, I did improve a little bit error reporting when SOAP faults are received: https://github.com/cloudescape/gowsdl/commit/8cdcd28bece8d0a3e52995c48cf04d3aaaf9e635

c4milo commented 9 years ago

@EntilZha these guys ran somewhat into a similar issue and worked around it: https://bitbucket.org/anacrolix/dms/commits/fbd23ce

c4milo commented 9 years ago

Regarding direction, I would start by reading how the Go xml package is currently dealing with namespaces to see if they can be easily defined at the top of the XML document, using prefixes. In other words, I would try to tackling this issue: https://code.google.com/p/go/issues/detail?id=6800

EntilZha commented 9 years ago

So just to test my understanding, the issue is the xmlns attribute on the getStations element? If it were not there, it would work correctly (it is not related to the "tns" in tns:getStations)?

So the "fix" would be figuring out a way to stick this at the top of the xml doc? <Envelope xmlns:tns="http://...">

c4milo commented 9 years ago

So just to test my understanding, the issue is the xmlns attribute on the getStations element?

Yes, it has to be replaced by a namespace prefix in the element instead.

So the "fix" would be figuring out a way to stick this at the top of the xml doc?

Yes, and setting the namespace with the prefix like so <tns:getStations></tns:getStation> instead of the URL repeated in each element of the XML document. The prefix will have to be auto-generated too, making sure it is unique within the document.

anacrolix commented 9 years ago

I look forward to seeing how you guys deal with this. As far as I'm concerned XML is an unfortunate necessity and poorly supported by the Go standard library. I also found that Go's "size optimizations" regarding " and ' caused problems with some decoders.

EntilZha commented 9 years ago

Any advice on where I might learn to get an idea of what I will need to do in order to accomplish that? (Reading through the prior link, wondering other resources to in general get knowledge required since I am not super savy on xml/soap prior to this)

c4milo commented 9 years ago

Hey @EntilZha, apologies for the very late response. So, in addition to what I previously mentioned regarding how to start tackling this issue, I would also suggest asking in the golang-dev mailing list. By the way, the issue was moved to https://github.com/golang/go/issues/6800.

EntilZha commented 9 years ago

I ended up on using Scala for this project in part for native support through existing Java libraries (in part from preference) and am too busy to work on this separately. If I end up having time in the future, I will take your advice though. Thanks

notzippy commented 9 years ago

I was able to resolve this issue by using another package : https://github.com/jteeuwen/go-pkg-xmlx , this package transformed the xml into an node struct, which I was then able to parse and reassign the name spaces in accordance to the headers. Then perform a SaveDoc and voila nicely marshelled document

c4milo commented 9 years ago

@notzippy, nice! do you think we could use that library in gowsdl?

notzippy commented 9 years ago

Hoping to, I am working through a solution... will let you know what I have in the form of a PR later

c4milo commented 9 years ago

ah, great! thank you!