Skip to content

fix: initial user signup fix#59

Merged
guacamole merged 1 commit intomasterfrom
cleanup
Nov 2, 2021
Merged

fix: initial user signup fix#59
guacamole merged 1 commit intomasterfrom
cleanup

Conversation

@jay-dee7
Copy link
Member

@jay-dee7 jay-dee7 commented Nov 2, 2021

We get the following error when the first user tries to register for beta:

{"error":"unexpected end of JSON input"}

This happens because we're trying to unmarshal nil byte slice into a struct with string slice in https://github.com/containerish/OpenRegistry/blob/master/cache/register_for_beta.go#L72

Missing return statement in the if condition above it caused this bug

Signed-off-by: jay-dee7 [email protected]

@jay-dee7 jay-dee7 requested a review from guacamole November 2, 2021 11:31
@jay-dee7 jay-dee7 self-assigned this Nov 2, 2021
Copy link
Member

@guacamole guacamole left a comment

Choose a reason for hiding this comment

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

LGTM 🥇

@guacamole guacamole merged commit ac6a928 into master Nov 2, 2021
@guacamole guacamole deleted the cleanup branch November 2, 2021 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants