Skip to content

Commit e7e5369

Browse files
authored
Fixed panic crash, subprocess hang. (#27)
1 parent 2465808 commit e7e5369

2 files changed

Lines changed: 3 additions & 11 deletions

File tree

README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ This project is the next-generation successor to [PyPacket](https://gihub.com/cc
88

99
## Release Notes
1010

11+
* 11/19/2022 ([1.0.3 release](https://github.com/cceremuga/ionosphere/releases/tag/v1.0.3))
12+
* Removed type logging to temporarily work around a panic/subprocess hang.
1113
* 11/15/2022 ([1.0.2 release](https://github.com/cceremuga/ionosphere/releases/tag/v1.0.2))
1214
* Fixed [an issue](https://github.com/cceremuga/ionosphere/issues/18) with config-supplied args for `rtl_fm` and `multimon-sg`.
1315
* Logging is now more verbose, including error output during the startup sequence for the underlying `rtl_fm`, `multimon-ng` processes.

framework/marshaler/marshaler.go

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -29,19 +29,9 @@ func Unmarshal(raw string) (*aprs.Packet, error) {
2929

3030
// Converts a packet to its reusable output format.
3131
func ToLogFormat(p *aprs.Packet) string {
32-
defer func() {
33-
if err := recover(); err != nil {
34-
log.Error(err, fmt.Sprintf(" (%s)", p.Raw))
35-
}
36-
}()
37-
38-
// Panic recovery above due to https://github.com/pd0mz/go-aprs/issues/5
39-
packetType := p.Payload.Type()
40-
41-
return fmt.Sprintf("%s -> %s [%s] (%f, %f) %s",
32+
return fmt.Sprintf("%s -> %s (%f, %f) %s",
4233
p.Src.Call,
4334
p.Dst.Call,
44-
packetTypeName(packetType),
4535
p.Position.Latitude,
4636
p.Position.Longitude,
4737
p.Comment,

0 commit comments

Comments
 (0)