Skip to content

Commit

Permalink
Merge pull request #5641 from onflow/petera/5640-update-string-concat…
Browse files Browse the repository at this point in the history
…-rpc-tracer1

[Network] Use string concatenation in rpc tracer ID
  • Loading branch information
peterargue authored Apr 9, 2024
2 parents 54a1eef + a224298 commit a203323
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions network/p2p/tracer/internal/rpc_sent_cache.go
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
package internal

import (
"fmt"

"github.com/rs/zerolog"

"github.com/onflow/flow-go/model/flow"
Expand Down Expand Up @@ -79,5 +77,5 @@ func (r *rpcSentCache) size() uint {
// Returns:
// - flow.Identifier: the entity ID.
func (r *rpcSentCache) rpcSentEntityID(messageId string, controlMsgType p2pmsg.ControlMessageType) flow.Identifier {
return flow.MakeIDFromFingerPrint([]byte(fmt.Sprintf("%s%s", messageId, controlMsgType)))
return flow.MakeIDFromFingerPrint([]byte(messageId + string(controlMsgType)))
}

0 comments on commit a203323

Please sign in to comment.