Skip to content

Add debug API in liblogosdelivery#3742

Open
Ivansete-status wants to merge 4 commits intomasterfrom
add-debug-api
Open

Add debug API in liblogosdelivery#3742
Ivansete-status wants to merge 4 commits intomasterfrom
add-debug-api

Conversation

@Ivansete-status
Copy link
Collaborator

Description

This introduces changes to implement the specs that have been added in this.

Changes

  • Add new functions in liblogosdelivery:
    • int logosdelivery_get_available_node_info_ids
    • int logosdelivery_get_node_info
    • int logosdelivery_get_available_configs
  • Update liblogosdelivery/examples/logosdelivery_example.c to use the new functions.

Related Issue

@github-actions
Copy link

github-actions bot commented Mar 2, 2026

You can find the image built from this PR at

quay.io/wakuorg/nwaku-pr:3742

Built from d64e9a9

ctx: ptr FFIContext[Waku],
callback: FFICallBack,
userData: pointer,
nodeInfoId: cstring,
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this 4th argument right? Looks like logosdelivery_get_available_configs is declared with 3 args?

int logosdelivery_get_available_configs(void *ctx,
                                    FFICallBack callback,
                                    void *userData);

Copy link
Collaborator Author

@Ivansete-status Ivansete-status Mar 2, 2026

Choose a reason for hiding this comment

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

Wow great catch! Thanks ! 🙌
Addressed in a0b6029

Copy link
Contributor

@fcecin fcecin left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

2 participants