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

add log output for virtualhosts. #1271

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@yannick
Contributor

yannick commented Sep 17, 2015

not sure if thats what you referred to @s-ludwig .
currently they are printed for each address family which makes them seem duplicates.
so it should also output the address family, however i was unsure on how to retrieve that.

add log output for virtualhosts.
currently they are printed for each address family which makes them seem duplicates.
so it should also output the address family, however i was unsure on how to retrieve that
@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Sep 22, 2015

Member

Yes, looks about like what I imagined. It would just have to happen at a different place for it to be always useful/correct. I'll have a second look later, as I don't remember the code details now.

Member

s-ludwig commented Sep 22, 2015

Yes, looks about like what I imagined. It would just have to happen at a different place for it to be always useful/correct. I'll have a second look later, as I don't remember the code details now.

@yannick

This comment has been minimized.

Show comment
Hide comment
@yannick

yannick Sep 22, 2015

Contributor

ok, i can fix it up once you give me a hint on what needs to be done

Contributor

yannick commented Sep 22, 2015

ok, i can fix it up once you give me a hint on what needs to be done

@s-ludwig s-ludwig closed this in 946b060 Jan 11, 2016

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Jan 11, 2016

Member

Finally got around to look into this. It was rather trivial to add, so I directly committed the change instead of wasting your time. Thanks anyway for pursuing this!

Member

s-ludwig commented Jan 11, 2016

Finally got around to look into this. It was rather trivial to add, so I directly committed the change instead of wasting your time. Thanks anyway for pursuing this!

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