Skip to content
This repository was archived by the owner on May 3, 2026. It is now read-only.

Fix LXMF client identity resolution before establishing links#193

Merged
brothercorvo merged 1 commit intomainfrom
corvo/fix-lxmf-link-setup-timeout-error
Oct 7, 2025
Merged

Fix LXMF client identity resolution before establishing links#193
brothercorvo merged 1 commit intomainfrom
corvo/fix-lxmf-link-setup-timeout-error

Conversation

@brothercorvo
Copy link
Copy Markdown
Contributor

Summary

  • add an asynchronous helper that waits for the server identity announce before building an LXMF link
  • update LXMF client tests to stub the identity resolver and cover the new behaviour
  • add unit coverage for identity wait success and timeout cases

Testing

  • pytest tests/test_client_extra.py

https://chatgpt.com/codex/tasks/task_e_68e56d3f51a48325a344c3bb3a4e9a6f

@brothercorvo
Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 👍

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting

@brothercorvo brothercorvo merged commit 12271a2 into main Oct 7, 2025
0 of 2 checks passed
@brothercorvo brothercorvo deleted the corvo/fix-lxmf-link-setup-timeout-error branch October 7, 2025 20:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant