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.

Sign up for GitHub

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

Jump to bottom

Validate UTF-8 strings in params #800

Merged
merged 2 commits into from
Jan 7, 2019
Merged

Validate UTF-8 strings in params #800

merged 2 commits into from
Jan 7, 2019

Conversation

bartekn
Copy link
Contributor

@bartekn bartekn commented Jan 2, 2019

Invalid UTF-8 byte sequence passed to Postgres triggers errors like:

invalid byte sequence for encoding "UTF8"

@bartekn bartekn added the horizon label Jan 2, 2019
@bartekn bartekn added this to the Horizon patch releases milestone Jan 2, 2019
@bartekn bartekn merged commit 408036e into master Jan 7, 2019
@bartekn bartekn deleted the check-utf8-params-valid branch January 7, 2019 15:10
oryband added a commit to kinecosystem/go that referenced this pull request Jan 28, 2019
@oryband
Merge WITH CONFLICTS tag 'horizon-v0.15.3' into update-to-0.15.3
0e13ee8 
- this commit included all files that have status "both modified" with conflicts
- in addition, i removed all files that had status "removed by them"
  since we didn't have any conflicts in these

* tag 'horizon-v0.15.3': (322 commits)
  Horizon 0.15.3
  Validate UTF-8 strings in params (stellar#800)
  Fix horizon reap. (stellar#782)
  Fix query checking (stellar#792)
  Fix run_tests (stellar#793)
  Update protocols/horizon README
  Horizon Documentation (stellar#763)
  Horizon 0.15.2
  Rebuild test scenarios and fix travis builds (stellar#787)
  Fix slow trades page query (stellar#785)
  Update docs with information about successful transactions
  Fix init-asset-stats command (stellar#775)
  Change `glide` to `dep` in Horizon README (stellar#784)
  Update asset.md (stellar#766)
  update path-finding docs (stellar#776)
  Update responses.md (stellar#765)
  Add "Postgres configuration" section in Horizon Admin Guide (stellar#773)
  Init asset stats command (stellar#764)
  Go 1.11 builds (stellar#762)
  Horizon 0.15.1
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomerweller tomerweller Awaiting requested review from tomerweller

@nikhilsaraf nikhilsaraf Awaiting requested review from nikhilsaraf

Assignees
No one assigned
Labels
horizon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bartekn