Skip to content

rust: update dependencies - #597

Open
tigroo wants to merge 4 commits into
masterfrom
fix_rust_advisories
Open

tigroo wants to merge 4 commits into
masterfrom
fix_rust_advisories

Conversation

@tigroo

@tigroo tigroo commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

Prepare the 3.2.1 version.

Changes

  • IoT3 SDK:
    • client: use the mqtt v5 client
    • exchange: use the mqtt v5 client
    • transport: use the mqtt v5 client
  • examples:
    • collector: use the mqtt v5 client
    • display: use the mqtt v5 client

Test

How to test

  1. Execute each example following the documentation into the README file.

Fix #599

Fix #600

@tigroo tigroo self-assigned this Sep 23, 2026
@tigroo tigroo added this to Kanban Sep 23, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in Kanban Sep 23, 2026
@tigroo
tigroo marked this pull request as draft September 23, 2026 14:23
@tigroo tigroo added the Rust Rust code label Sep 23, 2026
@tigroo tigroo moved this from Backlog to In progress in Kanban Sep 23, 2026
@tigroo
tigroo requested a review from nbuffon September 23, 2026 16:03
@tigroo
tigroo marked this pull request as ready for review September 23, 2026 16:06
Prepare the 3.2.1 version.

Signed-off-by: Frédéric Gardes <frederic.gardes@orange.com>
Prepare the 4.0.0 version.
Fix the websocket authentication.
Put a message size of 10000
(for the cpm's) by default).
Add a max_packet_size property.

Signed-off-by: Frédéric Gardes <frederic.gardes@orange.com>
Signed-off-by: Frédéric Gardes <frederic.gardes@orange.com>
Signed-off-by: Frédéric Gardes <frederic.gardes@orange.com>
@tigroo
tigroo force-pushed the fix_rust_advisories branch from 22ce3cf to 20d62f3 Compare September 28, 2026 13:22
}

let mut mqtt_options = self.mqtt_options.clone();
mqtt_options.set_client_id(format!("{}-{}", mqtt_options.client_id(), suffix));

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool news that we can now edit the client id without having to create a complete new struct !

You can then remove this method and directly update the client id in identity.rs:63

Comment thread rust/Cargo.toml
package = "rumqttc-v5-next"
version = "0.34.0"
default-features = false
features = ["websocket", "use-rustls-ring"]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why use-rustls-ring and not simply use-rustls ?

From what I found ring is based on BoringSSL which is not meant to be used by default.

This project was originally shared on GitHub in 2015 as an experiment. It was put on crates.io shortly to help other people with their experiments. It is an experiment.

Build it with: cargo run --example display_log_reader --features mobility\n\
(declared as required-features in Cargo.toml)"
);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With required-features declaration in Cargo.toml you already have an error at compile time if you don't set the feature in the build command

cargo build --example display_log_reader
error: target `display_log_reader` in package `libits-client` requires the features: `mobility`
Consider enabling them by passing, e.g., `--features="mobility"`

(Same for all examples)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Rust Rust code

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

use mqtt v5 client fix vulnerabilities

2 participants