wttw / aboutmyemail

Content for the aboutmy.email website
4 stars 1 forks source link

Panic in DNS resolution #51

Open wttw opened 1 month ago

wttw commented 1 month ago

panic: interface conversion: dns.RR is *dns.CNAME, not *dns.PTR Jul 31 09:56:53 aboutmy.email myemail[1115517]: goroutine 29257 [running]: Jul 31 09:56:53 aboutmy.email myemail[1115517]: turscar.ie/myemail/process.ProcessBasics.Process({}, {0x126ba08, 0xc001080690}, 0xc000e2d348, 0x0?) Jul 31 09:56:53 aboutmy.email myemail[1115517]: /Users/steve/dev/myemail/process/basics.go:152 +0xd0f Jul 31 09:56:53 aboutmy.email myemail[1115517]: turscar.ie/myemail/process.Process({_, _}, {{0xc0003318b0, 0xe}, {0xc0000facd8, 0x15}, {0xc0003545e0, 0x2, 0x2}, {0xc0003318d0, ...}, ...}, ...) Jul 31 09:56:53 aboutmy.email myemail[1115517]: /Users/steve/dev/myemail/process/process.go:44 +0x413 Jul 31 09:56:53 aboutmy.email myemail[1115517]: turscar.ie/myemail/web.(*Envelope).Close.func1() Jul 31 09:56:53 aboutmy.email myemail[1115517]: /Users/steve/dev/myemail/web/mailhook.go:106 +0x18f Jul 31 09:56:53 aboutmy.email myemail[1115517]: created by turscar.ie/myemail/web.(*Envelope).Close in goroutine 29247 Jul 31 09:56:53 aboutmy.email myemail[1115517]: /Users/steve/dev/myemail/web/mailhook.go:83 +0x7a9 Jul 31 09:56:53 aboutmy.email systemd[1]: myemail.service: Main process exited, code=exited, status=2/INVALIDARGUMENT

a) Why are we not catching panics and doing something more graceful with them?

b) Handle this case (possibly by ignoring it, if it's caused by a dangling CNAME).