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

Bugfix in CreateCustom #17

Merged
merged 4 commits into from
Mar 28, 2022
Merged

Bugfix in CreateCustom #17

merged 4 commits into from
Mar 28, 2022

Conversation

qqiao
Copy link
Owner

@qqiao qqiao commented Mar 28, 2022

Fixed a bug in create custom where when no expiresAt is specified,
the default expiresAt creation had two issues:

  • It used a go1.17 API time.UnixMicro, wihch breaks backward
    compatibility

  • The function was used wrongly as the 2nd param of time.Unix()

As a result of this, we have to issue this fix to ensure that the
1.4 stream still works with old version of the Go language. And we
also have to retract the v1.4.0 release.

Fixed a bug in create custom where when no expiresAt is specified,
the default expiresAt creation had two issues:

* It used a go1.17 API time.UnixMicro, wihch breaks backward
  compatibility

* The function was used wrongly as the 2nd param of time.Unix()

As a result of this, we have to issue this fix to ensure that the
1.4 stream still works with old version of the Go language. And we
also have to retract the v1.4.0 release.
@qqiao qqiao merged commit d25c6be into main Mar 28, 2022
@qqiao qqiao deleted the fix-unix-micro branch March 28, 2022 12:34
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.

None yet

1 participant