Skip to content

Tell the user the address that failed to connect. - #1007

Open
jeanbza wants to merge 1 commit into
zeromq:masterfrom
jeanbza:patch-1
Open

Tell the user the address that failed to connect.#1007
jeanbza wants to merge 1 commit into
zeromq:masterfrom
jeanbza:patch-1

Conversation

@jeanbza

@jeanbza jeanbza commented Dec 13, 2024

Copy link
Copy Markdown
Contributor

Current:

Caused by: org.zeromq.ZMQException: blah-blah : Address not available

Yes, but which address??

New error (I think?):

Caused by: org.zeromq.ZMQException: failed connecting to 0.0.0.0:5678: blah-blah : Address not available

Current:

```
Caused by: org.zeromq.ZMQException: blah-blah : Address not available
```

Yes, but _which_ address??

New error (I think?):

```
Caused by: org.zeromq.ZMQException: failed connecting to 0.0.0.0:5678: blah-blah : Address not available
```
@jeanbza
jeanbza marked this pull request as ready for review December 13, 2024 21:10
@jeanbza

jeanbza commented Dec 13, 2024

Copy link
Copy Markdown
Contributor Author

I'm new-ish to github, and have no clue if builds/tests are running on this PR. Sorry if I should have waited for them before sending out for review.

@fbacchella

Copy link
Copy Markdown
Contributor

I think you should use the name variable instead of addrStr

@jeanbza

jeanbza commented Oct 29, 2025

Copy link
Copy Markdown
Contributor Author

I think you should use the name variable instead of addrStr

Sorry, could you share more? What is name and what would it look like in comparison to addrStr?

@fbacchella

Copy link
Copy Markdown
Contributor

Just look at the argument of the function:

public InetSocketAddress resolve(String name, boolean ipv6, boolean local)

@paddor

paddor commented Aug 17, 2026

Copy link
Copy Markdown

This might be of interest to you: OMQ.java. Its TransportException stores operation, endpoint, and native detail. ConnectException, BindException, and NameResolutionException are subclasses of it. Tested here.

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.

3 participants