diff options
| author | Jeff Halter <868228+jhalter@users.noreply.github.com> | 2021-08-10 18:52:46 -0700 |
|---|---|---|
| committer | Jeff Halter <868228+jhalter@users.noreply.github.com> | 2021-08-10 13:52:46 -0700 |
| commit | 72dd37f1abb2b550aaaac48eac677403d5664797 (patch) | |
| tree | 5c431679475647f2f932f2934e6acd65090d2499 /hotline/client_conn.go | |
| parent | 9d41bcdf29778eab3253f8e31670baf64ad389bf (diff) | |
Add tests and clean up
Diffstat (limited to 'hotline/client_conn.go')
| -rw-r--r-- | hotline/client_conn.go | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/hotline/client_conn.go b/hotline/client_conn.go index eea3790..367b70f 100644 --- a/hotline/client_conn.go +++ b/hotline/client_conn.go @@ -29,7 +29,7 @@ type ClientConn struct { ID *[]byte Icon *[]byte Flags *[]byte - UserName *[]byte + UserName []byte Account *Account IdleTime *int Server *Server @@ -55,7 +55,7 @@ func (cc *ClientConn) handleTransaction(transaction *Transaction) error { if field.ID == nil { cc.Server.Logger.Infow( "Missing required field", - "Account", cc.Account.Login, "UserName", string(*cc.UserName), "RequestType", handler.Name, "FieldID", reqField.ID, + "Account", cc.Account.Login, "UserName", string(cc.UserName), "RequestType", handler.Name, "FieldID", reqField.ID, ) return nil } @@ -63,7 +63,7 @@ func (cc *ClientConn) handleTransaction(transaction *Transaction) error { if len(field.Data) < reqField.minLen { cc.Server.Logger.Infow( "Field does not meet minLen", - "Account", cc.Account.Login, "UserName", string(*cc.UserName), "RequestType", handler.Name, "FieldID", reqField.ID, + "Account", cc.Account.Login, "UserName", string(cc.UserName), "RequestType", handler.Name, "FieldID", reqField.ID, ) return nil } @@ -71,7 +71,7 @@ func (cc *ClientConn) handleTransaction(transaction *Transaction) error { if !authorize(cc.Account.Access, handler.Access) { cc.Server.Logger.Infow( "Unauthorized Action", - "Account", cc.Account.Login, "UserName", string(*cc.UserName), "RequestType", handler.Name, + "Account", cc.Account.Login, "UserName", string(cc.UserName), "RequestType", handler.Name, ) cc.Server.outbox <- cc.NewErrReply(transaction, handler.DenyMsg) @@ -81,7 +81,7 @@ func (cc *ClientConn) handleTransaction(transaction *Transaction) error { cc.Server.Logger.Infow( "Received Transaction", "login", cc.Account.Login, - "name", string(*cc.UserName), + "name", string(cc.UserName), "RequestType", handler.Name, ) @@ -95,7 +95,7 @@ func (cc *ClientConn) handleTransaction(transaction *Transaction) error { } else { cc.Server.Logger.Errorw( "Unimplemented transaction type received", - "UserName", string(*cc.UserName), "RequestID", requestNum, + "UserName", string(cc.UserName), "RequestID", requestNum, ) } @@ -114,7 +114,7 @@ func (cc *ClientConn) handleTransaction(transaction *Transaction) error { tranNotifyChangeUser, NewField(fieldUserID, *cc.ID), NewField(fieldUserFlags, *cc.Flags), - NewField(fieldUserName, *cc.UserName), + NewField(fieldUserName, cc.UserName), NewField(fieldUserIconID, *cc.Icon), ) |