While exploring the codebase, I noticed there is a //TODO: rename this file. comment at the top of pkg/agent/hooks/linux/comm.go.
Currently, the file is named comm.go, but it primarily contains functions (Get, GetDestinationInfo, GetTlsInfo, and SetTlsInfo) that handle network addresses and TLS information for the eBPF hooks. A name like network.go or network_info.go would be much more descriptive of what the file actually does.
Proposed Changes:
- Rename pkg/agent/hooks/linux/comm.go to pkg/agent/hooks/linux/network.go (or another name if the maintainers prefer).
- Remove the //TODO: rename this file. comment.
- Ensure that the //go:build linux directive remains at the top of the renamed file.
While exploring the codebase, I noticed there is a //TODO: rename this file. comment at the top of pkg/agent/hooks/linux/comm.go.
Currently, the file is named comm.go, but it primarily contains functions (Get, GetDestinationInfo, GetTlsInfo, and SetTlsInfo) that handle network addresses and TLS information for the eBPF hooks. A name like network.go or network_info.go would be much more descriptive of what the file actually does.
Proposed Changes: