@eyedeekay
&eche|on
&zzz
+Irc2PGuest3601
+RN
+StormyCloud
+acetone
+cumlord
+dr|z3d
+eche|off
+not_bob_afk
+orignal
+postman
+qend-irc2p
+snex
Arch2
Birdy
BubbRubb
Chrono
Daddy
Dann
DeltaOreo
Guest99
Irc2PGuest2010
Irc2PGuest20149
Irc2PGuest49068
Irc2PGuest61643
Irc2PGuest63942
Irc2PGuest74254
Irc2PGuest86777
Irc2PGuest95708
Irc2PGuest98006
Onn4l7h
Onn4|7h
Over1
Sisyphus
Sleepy
SlippyJoe
Stormycl1
T3s|4_
Teeed
Yotsu
aargh2
ac9f
anontor
b3t4f4c3___
dr4wd3
duanin2
duck
eyedeekay_bnc
gelleger1
leopold_
makoto
matean
n1
nilbog
onon_
poriori
profetikla
r00tobo
rapidash
shiver_
solidx66_
thetia
u5657
uop23ip
w8rabbit
x74a6
xHarr
eyedeekay
And off it goes
obscuratus
It's been a minute since I updated master.
obscuratus
If a router is *NOT* a FloodFill, is there any reason to handle a DLM for anything but their own RI?
obscuratus
Maybe it's healthy for the network as a fall-back? Or maybe just drop those early.
obscuratus
Of course, with a segmented netDb, it should be a moot point.
dr|z3d
obscuratus: I now drop lookup requests from other routers for anything other than our own RI if we're not a ff.
obscuratus
Thanks dr|z3d
dr|z3d
here's the code:
dr|z3d
} else if (!_context.netDb().floodfillEnabled() && !_facade.shouldBanLookup(dlm.getFrom(), dlm.getReplyTunnel())) {
dr|z3d
if (_log.shouldInfo()) {
dr|z3d
_log.warn("Dropping " + searchType + " lookup from [" + dlm.getFrom().toBase64().substring(0,6) + "] " +
dr|z3d
"for [" + dlm.getSearchKey().toBase64().substring(0,6) + "] -> " +
dr|z3d
"We are not a floodfill [TunnelId " + dlm.getReplyTunnel() + "]");
dr|z3d
} else if (_log.shouldWarn()) {
dr|z3d
_log.warn("Dropping " + searchType + " lookup from [" + dlm.getFrom().toBase64().substring(0,6) + "] " +
dr|z3d
"for [" + dlm.getSearchKey().toBase64().substring(0,6) + "] -> We are not a floodfill");
dr|z3d
}
dr|z3d
one sec, I'll get you a link for the entire context/file.
dr|z3d
git.skank.i2p/i2pplus/I2P.Plus/src/commit/46f542914f229c7a353cb2cfee093f96ac6173b3/router/java/src/net/i2p/router/networkdb/kademlia/FloodfillDatabaseLookupMessageHandler.java#L71
obscuratus
OK, I was confused. Canon is also dropping DLM when we're not a FF, unless for our own RI.
dr|z3d
sounds about right, obscuratus, I think eyedeekay followed my lead on that iirc.
obscuratus
Haha! This is embarassing. I'm the one who changed that.
dr|z3d
!