starred/geodns
1
0
Fork 0
mirror of https://github.com/abh/geodns.git synced 2026-04-27 03:45:53 +03:00
Code Issues 151 Projects Releases 5 Packages Wiki Activity

[GH-ISSUE #8] Closes UDP socket randomly(?) #6

New issue
Closed
opened 2026-02-28 14:15:49 +03:00 by kerem · 5 comments
kerem commented 2026-02-28 14:15:49 +03:00
Owner
Copy link

Originally created by @abh on GitHub (Aug 31, 2012).
Original GitHub issue: https://github.com/abh/geodns/issues/8

This is a followup to miekg/dns#21 (which really was about something else).

I ran dnsbench against the geodns server today to reproduce the "hang" where it just stops listening on UDP. After 4 hours of 1000qps it was still working fine and it's "only" doubled memory usage from 15MB to 30MB.

Running lsof on the process shows it's doesn't have the UDP server open anymore.

I've added some monitoring code in 70b6e4e413 and an attempt at logging if ListenAndServe returns: 98020a3f63

Originally created by @abh on GitHub (Aug 31, 2012). Original GitHub issue: https://github.com/abh/geodns/issues/8 This is a followup to miekg/dns#21 (which really was about something else). I ran dnsbench against the [geodns](http://github.com/abh/geodns) server today to reproduce the "hang" where it just stops listening on UDP. After 4 hours of 1000qps it was still working fine and it's "only" doubled memory usage from 15MB to 30MB. Running `lsof` on the process shows it's doesn't have the UDP server open anymore. I've added some monitoring code in 70b6e4e413f8751678c2999e8a6d124c954676d3 and an attempt at logging if ListenAndServe returns: 98020a3f63515b4a
kerem closed this issue 2026-02-28 14:15:49 +03:00
kerem commented 2026-02-28 14:15:50 +03:00
Author
Owner
Copy link

@miekg commented on GitHub (Aug 31, 2012):

Can it be that ServeUDP() is reading to much and returning. server.go lines 274-277. I could change that into a continue instead of a return...?

<!-- gh-comment-id:8184668 --> @miekg commented on GitHub (Aug 31, 2012): Can it be that ServeUDP() is reading to much and returning. server.go lines 274-277. I could change that into a continue instead of a return...?
kerem commented 2026-02-28 14:15:50 +03:00
Author
Owner
Copy link

@abh commented on GitHub (Aug 31, 2012):

Maybe start with just making it a (loud) log warning and then I can see if it shows up. The server has enough different clients that it'll get the occasional completely non-sense request.

<!-- gh-comment-id:8184750 --> @abh commented on GitHub (Aug 31, 2012): Maybe start with just making it a (loud) log warning and then I can see if it shows up. The server has enough different clients that it'll get the occasional completely non-sense request.
kerem commented 2026-02-28 14:15:50 +03:00
Author
Owner
Copy link

@miekg commented on GitHub (Aug 31, 2012):

Yeah, but I'm changing that anyhow. Because it makes the listener quit when it gets a weird packet - not something you want.

<!-- gh-comment-id:8184772 --> @miekg commented on GitHub (Aug 31, 2012): Yeah, but I'm changing that anyhow. Because it makes the listener quit when it gets a weird packet - not something you want.
kerem commented 2026-02-28 14:15:50 +03:00
Author
Owner
Copy link

@abh commented on GitHub (Aug 31, 2012):

I confirmed that the problem was ListenAndServe returning. (I didn't get to upgrade last night before going to sleep).

I upgraded the test server now to use the latest dns version. I'm planning to move a couple more servers to the Go version over the weekend. Thanks for all the help!

<!-- gh-comment-id:8202924 --> @abh commented on GitHub (Aug 31, 2012): I confirmed that the problem was ListenAndServe returning. (I didn't get to upgrade last night before going to sleep). I upgraded the test server now to use the latest dns version. I'm planning to move a couple more servers to the Go version over the weekend. Thanks for all the help!
kerem commented 2026-02-28 14:15:50 +03:00
Author
Owner
Copy link

@miekg commented on GitHub (Aug 31, 2012):

No problem. I have fun helping :) See latest pull request :) Actually debugging with dnsperf as we speak. I'll let it run for about 4 hours.

(Thanks for the close #N tip!)

<!-- gh-comment-id:8203167 --> @miekg commented on GitHub (Aug 31, 2012): No problem. I have fun helping :) See latest pull request :) Actually debugging with dnsperf as we speak. I'll let it run for about 4 hours. (Thanks for the close #N tip!)
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
copilot/fix-102
avro
dependabot/go_modules/golang.org/x/net-0.7.0
master
godep
docs
geoip6
influxdb
targeting
edns-subnet
v3.4.1
v3.4.0
v3.3.3
v3.3.2
v3.3.1
v3.3.0
v3.2.3
v3.2.2
v3.2.1
v3.2.0
v3.1.0
v3.1.0-rc1
v3.0.2
v3.0.1
2.7.0
2.6.0
2.5.0
2.4.0
2.4.4
2.4.1
2.2.8
2.2.7
2.2.6
2.2.5
2.2.4
2.2.3
2.2.2
2.2.1
Labels
Clear labels   
bug

   
bug

   
enhancement

   
pull-request

Mirrored from GitHub Pull Request

  
question
No labels
bug
bug
enhancement
pull-request
question
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/geodns#6
No description provided.