Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't accept consumer connection until provider connection has been established #1

Closed
andreashuber-lawo opened this issue Sep 24, 2015 · 0 comments
Closed

Don't accept consumer connection until provider connection has been established #1

andreashuber-lawo opened this issue Sep 24, 2015 · 0 comments
Assignees
@andreashuber-lawo
Labels
bug Glow Analyzer Proxy
Milestone
v1.0

Comments

@andreashuber-lawo
Copy link
Contributor

Currently, GlowAnalyzerProxy first accepts the connection from the consumer before it attempts to connect to the provider. This behavior is incorrect for the case when the provider does not answer the connection request.
@andreashuber-lawo andreashuber-lawo self-assigned this Sep 24, 2015
@andreashuber-lawo andreashuber-lawo added this to the v1.0 milestone Oct 11, 2015
andreashuber-lawo added a commit that referenced this issue Oct 29, 2015
@andreashuber-lawo
Add direction parameter to IS101Logger.LogException
0da80cb 
This will allow us to log whether the exception originated with the provider or the consumer.

References #1
andreashuber-lawo added a commit that referenced this issue Oct 29, 2015
@andreashuber-lawo
Introduce constants for the direction strings
f02f852 
References #1
andreashuber-lawo added a commit that referenced this issue Oct 29, 2015
@andreashuber-lawo
Fix StyleCop issues
13d4741 
References #1
andreashuber-lawo added a commit that referenced this issue Oct 29, 2015
@andreashuber-lawo
Move await statements into catch and finally blocks
7b9eab8 
References #1
andreashuber-lawo added a commit that referenced this issue Oct 29, 2015
@andreashuber-lawo
Add direction information to exception logging
15f8f1f 
References #1
andreashuber-lawo added a commit that referenced this issue Oct 30, 2015
@andreashuber-lawo
Count connection attempts rather than successful connections
00040f2 
References #1
andreashuber-lawo added a commit that referenced this issue Oct 30, 2015
@andreashuber-lawo
Limit the pending connection request queue length to 1
9b00258 
References #1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andreashuber-lawo andreashuber-lawo

Labels
bug Glow Analyzer Proxy
Projects
None yet
Milestone
v1.0
Development

No branches or pull requests
1 participant
@andreashuber-lawo