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

Add transport command to leshan-client-demo #1522

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sbernard31
Copy link
Contributor

This is experimentation about adding a transport command to leshan-client-demo to be able to configure transport layer.

This looks like :

Usage: leshan-client-demo transport [-h] [COMMAND]
Configure Leshan transport layer. Can be used to add, remove or use different
implementation of supported LWM2M transport.

By default, Californium library is used for coap/coaps. Some Californium
parameters can be tweaked in Californium3.client.properties file.
Default behavior is like using :
transport californium coap coaps

More examples :
coaps support only based on Californium library :
transport californium coaps

coap support based on java-coap library :
transport java-coap coap

coaps support based on Californium library and coap support based java-coap
library:
transport californium coaps java-coap coap

Launch transport [californium|java-coap] -h for more details.

  -h, --help   Display help information.
Commands:
  californium  Configure transport based on Californium.
  java-coap    Configure transport based on java-coap.

This could sound a bit overkill but this will allow to have specific option for each transport.
E.g. connection id option should probably be moved have a transport californium coaps option

@sbernard31 sbernard31 marked this pull request as draft October 3, 2023 15:32
@sbernard31 sbernard31 changed the base branch from master to java_coap_client October 3, 2023 15:33
@eclipse-leshan eclipse-leshan deleted a comment from github-actions bot Oct 3, 2023
@eclipse-leshan eclipse-leshan deleted a comment from github-actions bot Oct 3, 2023
@sbernard31
Copy link
Contributor Author

❌ Bad news, picocli behavior is not what I expected and so this way to go will not work... 😬

See : remkop/picocli#2133 for more details.

For now I didn't find any alternative... 😞

@sbernard31 sbernard31 force-pushed the java_coap_client branch 3 times, most recently from b6572f1 to 85cfde9 Compare October 19, 2023 15:45
@sbernard31 sbernard31 changed the base branch from java_coap_client to master October 19, 2023 16:27
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