Skip to content

fix(core/choice): log on dial and fix listen log#4164

Merged
mergify[bot] merged 2 commits intolibp2p:masterfrom
mxinden:choice-listen-dial
Jul 10, 2023
Merged

fix(core/choice): log on dial and fix listen log#4164
mergify[bot] merged 2 commits intolibp2p:masterfrom
mxinden:choice-listen-dial

Conversation

@mxinden
Copy link
Copy Markdown
Member

@mxinden mxinden commented Jul 10, 2023

Description

  • Fixes typo in <OrTransport as Transport>::listen_on using the word "dial" instead of "listen_on".
  • Adds logging to <OrTransport as Transport>::dial.

Follow-up to #4133

Notes & open questions

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • A changelog entry has been made in the appropriate crates

- Fixes typo in `<OrTransport as Transport>::listen_on` using the word "dial" instead of "listen_on".
- Adds logging to `<OrTransport as Transport>::dial`.

Follow-up to libp2p#4133
@mxinden mxinden added send-it trivial Marks PRs which are considered trivial and don't need approval from another maintainer. labels Jul 10, 2023
@mergify mergify bot merged commit 2671375 into libp2p:master Jul 10, 2023
thomaseizinger pushed a commit that referenced this pull request Aug 20, 2023
- Fixes typo in `<OrTransport as Transport>::listen_on` using the word "dial" instead of "listen_on".
- Adds logging to `<OrTransport as Transport>::dial`.

Follow-up to #4133

Pull-Request: #4164.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

send-it trivial Marks PRs which are considered trivial and don't need approval from another maintainer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant