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

Fix character-set issues #108

Merged
merged 9 commits into from
Jan 8, 2019
Merged

Fix character-set issues #108

merged 9 commits into from
Jan 8, 2019

Conversation

digitalsparky
Copy link
Contributor

Hi there, I was playing around with your tool, however, it crashes out when there is a character set other than UTF-8.

This patch resolves that issue.
I've included the package dependencies using 'dep' (gopkg lock files included).

@fiorix
Copy link
Owner

fiorix commented Oct 29, 2018

I guess this is the PR you cancelled due to conflicts. You ok with re-opening this?

@digitalsparky digitalsparky reopened this Oct 29, 2018
@digitalsparky
Copy link
Contributor Author

Yup, there you go :).

@mavogel
Copy link

mavogel commented Nov 2, 2018

@digitalsparky thx for the PR. Works fine for non-UTF8 wsdls like https://www.demo.ilias.de/webservice/soap/server.php?wsdl

@fiorix any ETA when this gets merged. I built it locally atm

@digitalsparky
Copy link
Contributor Author

Pull request updated so you can easily merge into upstream (reverts forked namespace changes)

@digitalsparky
Copy link
Contributor Author

ping @fiorix

@digitalsparky
Copy link
Contributor Author

note: needs to built with GO111MODULE=on env var

@fiorix
Copy link
Owner

fiorix commented Jan 7, 2019

CI still failing. Please fix the tests and I'll be happy to merge.

@fiorix
Copy link
Owner

fiorix commented Jan 7, 2019

Still failing on Go 1.9 for obvious reasons. Gotta drop older versions and leave Go 1.11 and tip.

@digitalsparky
Copy link
Contributor Author

Fixed, was using go mod but that's not supported in go 1.9, changed to dep.
Also imports pull request #112 as I ran into it when building.

@digitalsparky
Copy link
Contributor Author

Recommend changing to go mod once you deprecate 1.9 support.

@fiorix fiorix merged commit 19735c0 into fiorix:master Jan 8, 2019
@fiorix
Copy link
Owner

fiorix commented Jan 8, 2019

👍

I squashed your commits and merged.

@digitalsparky
Copy link
Contributor Author

Thanks, have a great day!

This was referenced Jan 9, 2019
jhedev pushed a commit to grid-x/wsdl2go that referenced this pull request Jan 16, 2019
* Fix character-set issues

* Add decoder charset compliance to soap client

* Update travis ci file to force env var for build and path to golint
GnusinPavel pushed a commit to smartway-dev/wsdl2go that referenced this pull request Mar 18, 2019
* Fix character-set issues

* Add decoder charset compliance to soap client

* Update travis ci file to force env var for build and path to golint
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.

3 participants