eclipse-californium / californium

CoAP/DTLS Java Implementation
https://www.eclipse.org/californium/
Other
730 stars 367 forks source link

DatagramFilterExtended#onDrop called with not "full" Record #2171

Closed Wisien999 closed 1 year ago

Wisien999 commented 1 year ago

I noticed there is a case where org.eclipse.californium.scandium.DatagramFilterExtended#onDrop(org.eclipse.californium.scandium.dtls.Record) is called with peerAddress set to null. It can cause some problems (It did in my organization). Seems like this is the only such situation. Would it be possible to add firstRecord.setAddress(peerAddress, router) one line above? I should solve the issue. Also as I would consider it a bug is it possible to release this fix quickly with patch version change?

boaks commented 1 year ago

I'm on the bugfix. If successful, I schedule a bugfix release 3.9.1 for Thursday.

boaks commented 1 year ago

If possible, please verify the bugfix.

Wisien999 commented 1 year ago

Sorry for the late response. That's exactly what I meant. Should work. Thank you.

boaks commented 1 year ago

OK, then I schedule a Bugfix Release 3.9.1 for next Thursday.

boaks commented 1 year ago

See #2181