-
Notifications
You must be signed in to change notification settings - Fork 58
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
Exchange creation failing if local address overridden #29
Comments
Also, this is causing the acceptance tests to hang. |
I think the test is wrong, since Rabbit needs to use the endpoint name during initalization we can't change it after: I say we modify the test to use IWantToRunBeforeConfiguration instead |
We can try, but not sure it will fit the order for ASB's case. |
@yvesgoeleven ok, lets see if this breaks ASB instead :( That said overriding local address seems like a think we want to do as early as possible? |
Not sure what the current situation is, but in the past getting timing for this setting right was a big pita as it was initialized rather late (and changed quite a few times over the past years) |
Have just tried the fix with |
Hmm, pretty sure it did work on the buildserver? On Tue, Apr 29, 2014 at 9:46 AM, John Simons notifications@github.comwrote:
|
Nope it fails locally for me, try it on your machine On Tuesday, April 29, 2014, Andreas Öhlund notifications@github.com wrote:
|
@andreasohlund did u test this? This is the exception I get locally:
|
Works locally and on buildserve, closing as can't repro |
@andreasohlund I find that very strange, since that test is still marked as Line 13 in 919fc61
|
BTW, I ran it again and still same error |
Doh!! Good catch On Sun, Nov 30, 2014 at 11:36 PM, John Simons notifications@github.com
|
Fixed by df67c3c |
Who's affected
All users overriding the local address of the endpoint
Symptoms
The endpoint fails when subscribing throwing the following exception:
The text was updated successfully, but these errors were encountered: