temporalio / roadrunner-temporal

Temporal PHP-SDK Host Process plugin for Roadrunner

MIT License

22 stars 8 forks source link

fix: update proto marshaler from gogo to std #469

Closed rustatian closed 8 months ago

rustatian commented 8 months ago

Reason for This PR

Replay WF error when using old gogo-proto encoded history.

Description of Changes

Remove gogo proto, update with the standard (google/protobuf) json marshaler.

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.] [Reviewer TODO: Verify that these criteria are met. Request changes if not]

[x] All commits in this PR are signed (git commit -s).
[x] The reason for this PR is clearly provided (issue no. or explanation).
[x] The description of changes is clear and encompassing.
[x] Any required documentation changes (code and docs) are included in this PR.
[x] Any user-facing changes are mentioned in CHANGELOG.md.
[x] All added/changed functionality is tested.