Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Experiments some LWM2M over coap+tcp based on californium #1312

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sbernard31
Copy link
Contributor

@sbernard31 sbernard31 commented Sep 14, 2022

This is an experimental work based on #1318 #1323 and #1336 and not mature coap+tcp implementation of californium.
(it aims to implement #1047)

@sbernard31
Copy link
Contributor Author

I rebase this PR on new cleaner version of abstraction layer (#1318 , #1323)

@sbernard31
Copy link
Contributor Author

Sometime I face this kind of exception :

org.eclipse.leshan.core.request.exception.UnconnectedPeerException: Unable to send request coap://127.0.0.1:56124/3/0 : peer is not connected
	at org.eclipse.leshan.core.californium.DefaultExceptionTranslator.translate(DefaultExceptionTranslator.java:28)
	at org.eclipse.leshan.core.californium.CoapSyncRequestObserver.onSendError(CoapSyncRequestObserver.java:100)
	at org.eclipse.californium.core.coap.Message.setSendError(Message.java:1029)
	at org.eclipse.californium.core.coap.Request.setSendError(Request.java:1032)
	at org.eclipse.californium.core.network.CoapEndpoint$SendingCallback.onError(CoapEndpoint.java:1299)
	at org.eclipse.californium.elements.RawData.onError(RawData.java:314)
	at org.eclipse.californium.elements.tcp.netty.TcpServerConnector.send(TcpServerConnector.java:200)
	at org.eclipse.californium.core.network.CoapEndpoint$OutboxImpl.sendRequest(CoapEndpoint.java:924)
	at org.eclipse.californium.core.network.stack.BaseCoapStack$StackBottomAdapter.sendRequest(BaseCoapStack.java:231)
	at org.eclipse.californium.core.network.stack.AbstractLayer.sendRequest(AbstractLayer.java:69)
	at org.eclipse.californium.core.network.stack.BlockwiseLayer.sendRequest(BlockwiseLayer.java:498)
	at org.eclipse.californium.core.network.stack.TcpObserveLayer.sendRequest(TcpObserveLayer.java:57)
	at org.eclipse.californium.core.network.stack.AbstractLayer.sendRequest(AbstractLayer.java:69)
	at org.eclipse.californium.core.network.stack.TcpExchangeCleanupLayer.sendRequest(TcpExchangeCleanupLayer.java:45)
	at org.eclipse.californium.core.network.stack.BaseCoapStack$StackTopAdapter.sendRequest(BaseCoapStack.java:188)
	at org.eclipse.californium.core.network.stack.BaseCoapStack.sendRequest(BaseCoapStack.java:87)
	at org.eclipse.californium.core.network.CoapEndpoint$3.run(CoapEndpoint.java:701)
	at org.eclipse.californium.elements.util.SerialExecutor$1.run(SerialExecutor.java:293)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: org.eclipse.californium.elements.exception.EndpointUnconnectedException: TCP client not connected!
	... 15 common frames omitted

I didn't investigate yet.

@sbernard31
Copy link
Contributor Author

About UnconnectedPeerException, my investigation at : eclipse-californium/californium#2081

@sbernard31
Copy link
Contributor Author

(The UnconnectedPeerException issue should be "fixed" in last commit in an acceptable way for an experimental feature : eclipse-californium/californium#2081 (comment))

@sbernard31 sbernard31 changed the base branch from endpoint_bootstrap to master January 26, 2023 16:27
@sbernard31
Copy link
Contributor Author

At least at short/mid term, there will not be more work on this (see #1047 (comment))
But we can keep the PR in case, one day someone succeed to add coap+tcp to Californium.

@sbernard31 sbernard31 force-pushed the master branch 2 times, most recently from 8e7cd35 to 2efb924 Compare November 8, 2023 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant