Skip to content
This repository has been archived by the owner on Dec 18, 2018. It is now read-only.

Add a public HttpConnection.ConnectionId #963

Closed
KPixel opened this issue Sep 28, 2017 · 2 comments
Closed

Add a public HttpConnection.ConnectionId #963

KPixel opened this issue Sep 28, 2017 · 2 comments

Comments

@KPixel
Copy link

KPixel commented Sep 28, 2017

Would it be ok to make the ConnectionId visible outside of HttpConnection?
Something like:

public string ConnectionId => _connectionId;

It could also be exposed through the Features property if you don't want to expose it directly like that.

Reason: When using the Sockets library, the client has no way of knowing its own ConnectionId.
I am working on a scenario where the client needs to send it to a notification service (think of it as a decentralized presence system).

@davidfowl
Copy link
Member

davidfowl commented Sep 28, 2017

No because the client will not always have the connection id. See #788 (comment) (and generally the overall thread about this)

Also #932

@KPixel
Copy link
Author

KPixel commented Sep 28, 2017

My bad. I didn't expect this issue to have already been discussed, so I didn't search for it.

I am going to continue this discussion here: #961.

@KPixel KPixel closed this as completed Sep 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants