Skip to content

Allow Extraction of Client TLS Certificate (CN/SAN) at rmqtt-net or Plugin Layer in MQTT Context #313

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.

Sign up for GitHub

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

Jump to bottom
Merged
bittcrafter merged 20 commits into from
Oct 16, 2025
Merged

Allow Extraction of Client TLS Certificate (CN/SAN) at rmqtt-net or Plugin Layer in MQTT Context #313

bittcrafter merged 20 commits into from
Oct 16, 2025

Conversation

@deluxetiky
Copy link
Contributor

@deluxetiky deluxetiky commented Oct 12, 2025

Discussion Link

#312

@bittcrafter
Copy link
Collaborator

bittcrafter commented Oct 14, 2025

Thanks a lot for your contribution! 🙌
I really appreciate your work on this PR. I’ll carefully review the changes and consider merging them soon.

@bittcrafter
Copy link
Collaborator

bittcrafter commented Oct 14, 2025

Suggest merging the mqtt() and mqtt_tls() methods into a single unified implementation for a cleaner structure.

@deluxetiky
Copy link
Contributor Author

deluxetiky commented Oct 15, 2025

Suggest merging the mqtt() and mqtt_tls() methods into a single unified implementation for a cleaner structure.

I did merged both methods and fixed one of the CI errors related to this pr. Could you please check.

deluxetiky and others added 3 commits October 16, 2025 02:06
@deluxetiky
Feature config updates
08bd2a9
@bittcrafter
Update builder.rs
075af0b
@bittcrafter
Refactor mqtt function to use self.cert_info
1b5b15e 
Removed TlsCertExtractor dependency and updated mqtt function to use cert_info from self.
@bittcrafter bittcrafter merged commit f608d76 into rmqtt:master Oct 16, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@deluxetiky @bittcrafter