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

listenTCP with libasync causes connection.localAddress.family to be AF_UNSPEC #961

Closed
GeorgeSapkin opened this Issue Jan 21, 2015 · 3 comments

Comments

Projects
None yet
4 participants
@GeorgeSapkin
Contributor

GeorgeSapkin commented Jan 21, 2015

In the following example NetworkAddress.toAddressString() fails with called for invalid address family (family is AF_UNSPEC) when switching to libasync driver. Default libevent driver works as expected and I would expect the same behavior from libasync. Reproducible on both Linux and Windows with vibe 0.7.22 and libasync 0.6.6.

listenTCP(port,
    (connection)
    {
        auto addr = connection.localAddress.toAddressString;
    });

Not sure where this should be reported.

@GeorgeSapkin GeorgeSapkin changed the title from [external bug] listenTCP with libasync causes connection.localAddress.family to be AF_UNSPEC to listenTCP with libasync causes connection.localAddress.family to be AF_UNSPEC Jan 21, 2015

@Geod24

This comment has been minimized.

Show comment
Hide comment
@Geod24

Geod24 Jan 21, 2015

Contributor

Ping @etcimon :)

Contributor

Geod24 commented Jan 21, 2015

Ping @etcimon :)

etcimon added a commit to etcimon/vibe.d that referenced this issue Jan 21, 2015

etcimon added a commit to etcimon/vibe.d that referenced this issue Jan 21, 2015

@etcimon

This comment has been minimized.

Show comment
Hide comment
@etcimon

etcimon Jan 21, 2015

Contributor

My bad, thanks for catching it!

Contributor

etcimon commented Jan 21, 2015

My bad, thanks for catching it!

s-ludwig added a commit that referenced this issue Feb 21, 2015

Merge pull request #962 from etcimon/fix-libasync
Local address not set correctly - Fixes issue #961
@etcimon

This comment has been minimized.

Show comment
Hide comment
@etcimon

etcimon Mar 25, 2015

Contributor

This can be closed now

Contributor

etcimon commented Mar 25, 2015

This can be closed now

@s-ludwig s-ludwig closed this Mar 25, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment