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 incorrect decoding of array when input contains more elements #8

Closed
wants to merge 1 commit into from
Closed

Fix incorrect decoding of array when input contains more elements #8

wants to merge 1 commit into from

Conversation

nebolsin
Copy link

@nebolsin nebolsin commented Sep 3, 2018

Upstream issue: stellar/go#374.

Originally planned to work around it on stellar/go side (with stellar/go#597), but looks like the actual fix belongs here.

@nebolsin
Copy link
Author

/cc @nullstyle

@bartekn
Copy link

bartekn commented Sep 13, 2018

@nebolsin great catch! Since @nullstyle is on sabbatical leave, let me organize a fork of this repo in stellar org and we will move your PR there. Then I will update go projects to use a new location.

@nullstyle
Copy link
Owner

Sorry, I havent checked in with github in a couple days. I think moving to a stellar managed fork makes the most sense as well @bartekn, thanks for arranging that.

@bartekn
Copy link

bartekn commented Sep 17, 2018

Cherry-picked into: stellar@0bc96f3. @nebolsin this can be closed.

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