Skip to content
This repository has been archived by the owner on Sep 11, 2023. It is now read-only.

Update Pitaya version & misc improvements #23

Merged
merged 16 commits into from
Nov 3, 2021

Conversation

CaioIcy
Copy link
Contributor

@CaioIcy CaioIcy commented Apr 1, 2021

The main updates are:

  • Updates Pitaya from 0.14.3 to 1.1.5
  • Fixes example tests (JSON & Protobuffer)
  • Improved logging
  • Add server.requestTimeout config
  • Spec now validates some fields before running

CaioIcy added 15 commits March 24, 2021 21:28
Error was the following:
go: github.com/topfreegames/pitaya@v0.14.3 requires
	github.com/jhump/protoreflect@v0.0.0-20181126200011-b5ebaabe89bf requires
	google.golang.org/genproto@v0.0.0-20170818100345-ee236bd376b0: invalid pseudo-version: does not match version-control timestamp (2017-08-18T01:03:45Z)
Also removes TLS option, the client will try to connect in every way
until it succedes.
In some cases where the spec couldn't finish this failure wouldn't be
logged at all
To match Request behaviour so we can also build a NewExpectError
bot/pitaya_client.go Show resolved Hide resolved
bot/pitaya_client.go Show resolved Hide resolved
bot/sequential.go Show resolved Hide resolved
runner/runner.go Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@henrod
Copy link
Contributor

henrod commented Apr 7, 2021

LGTM!

@CaioIcy
Copy link
Contributor Author

CaioIcy commented Apr 8, 2021

Thanks! No write access here, can you merge?

@henriqueoelze henriqueoelze self-requested a review November 3, 2021 19:26
Copy link
Contributor

@henriqueoelze henriqueoelze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aligned with @henrod and we can merge it.

@henriqueoelze henriqueoelze merged commit 7799507 into topfreegames:master Nov 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants