Skip to content

Commit cbeffa3

Browse files
committed
fix: add null checks for remoteSocketAddress in WebsocketServerServiceImp
Updated `WebsocketServerServiceImp` to safely handle potential null values for `remoteSocketAddress` when retrieving client address and port. Ref: #72
1 parent 30a6021 commit cbeffa3

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

data/src/main/java/github/umer0586/sensorserver/data/service/websocket/WebsocketServerServiceImp.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -230,8 +230,8 @@ class WebsocketServerServiceImp : Service(), WebsocketServerService {
230230
val websocketClientsList = mutableListOf<WebsocketClient>()
231231
webSockets.forEach { websocket ->
232232

233-
val address = websocket.remoteSocketAddress.address.hostName
234-
val port = websocket.remoteSocketAddress.port
233+
val address = websocket.remoteSocketAddress?.address?.hostName ?: return@forEach
234+
val port = websocket.remoteSocketAddress?.port ?: return@forEach
235235

236236
if(websocket.getAttachment<Any>() is Sensor){
237237
val deviceSensor = (websocket.getAttachment<Any>() as Sensor).toDeviceSensor()

0 commit comments

Comments
 (0)