Unverified Commit 8e8a0fe4 authored by Luke Gehorsam's avatar Luke Gehorsam Committed by GitHub
Browse files

improve Steam logs on GetSteamProfile error (#1120)

parent 56814441

server/core_authenticate.go

+1 −1
Original line number Diff line number Diff line
@@ -831,7 +831,7 @@ func importSteamFriends(ctx context.Context, logger *zap.Logger, db *sql.DB, mes 


	steamProfiles, err := client.GetSteamFriends(ctx, publisherKey, steamId)

	if err != nil {

		logger.Info("Could not import Steam friends.", zap.Error(err))

		logger.Error("Could not import Steam friends.", zap.Error(err))

		return status.Error(codes.Unauthenticated, "Could not authenticate Steam profile.")

	}

social/social.go

+3 −0
Original line number Diff line number Diff line
@@ -626,12 +626,15 @@ func (c *Client) GetSteamProfile(ctx context.Context, publisherKey string, appID 
	var profileWrapper SteamProfileWrapper

	err := c.request(ctx, "steam profile", path, nil, &profileWrapper)

	if err != nil {

		c.logger.Debug("Error requesting Steam profile", zap.Error(err))

		return nil, err

	}

	if profileWrapper.Response.Error != nil {

		c.logger.Debug("Error returned from Steam after requesting Steam profile", zap.String("errorDescription", profileWrapper.Response.Error.ErrorDesc), zap.Int("errorCode", profileWrapper.Response.Error.ErrorCode))

		return nil, fmt.Errorf("%v, %v", profileWrapper.Response.Error.ErrorDesc, profileWrapper.Response.Error.ErrorCode)

	}

	if profileWrapper.Response.Params == nil {

		c.logger.Debug("No profile returned from Steam after requesting Steam profile")

		return nil, errors.New("no steam profile")

	}

	return profileWrapper.Response.Params, nil