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

Client library extended auth handler #2817

Open
wants to merge 805 commits into
base: master
Choose a base branch
from

Conversation

butleger
Copy link

Hello, this is my first pull-request.

I want to add functionality to extended auth in client library. As i see there is no C lib with this feature. Tests are not passed(, it because of function packet__get_next_out in packet_mosq.c, but i dont know why it so.

ralight and others added 30 commits November 1, 2022 15:08
This means that if set to 10, for example, updates will be sent at times
matching x0 seconds. Previously update intervals were aligned to the
time the broker was started.
Signed-off-by: Kai Buschulte <[email protected]>
sub_count is not the actual number of subscriptions.
Renamed it to subs_capacity, as it keeps the value of allocated
mosquitto__client_sub elements stored in the subs member of the
mosquitto struct.

Signed-off-by: Kai Buschulte <[email protected]>
This is the actual number of subscriptions.

Signed-off-by: Kai Buschulte <[email protected]>
Signed-off-by: Kai Buschulte <[email protected]>
…fig-error-tests

Extend broker tests for config errors
`mosquitto_property_read_binary/string/string_pair` will now set the
name/value parameter to NULL if the binary/string is empty. This aligns the
behaviour with other property functions.

Closes eclipse#2648. Thanks to Alexander Fiebig.
@butleger butleger force-pushed the client_library_extended_auth_handler branch from df2a3f7 to e93f334 Compare May 26, 2023 16:01
@butleger butleger force-pushed the client_library_extended_auth_handler branch from e93f334 to fb21a52 Compare May 27, 2023 18:01
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.

None yet

8 participants