-
Notifications
You must be signed in to change notification settings - Fork 5
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
fix: Update streaming connection URL when config received #891
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
38a65ae
to
745cb0c
Compare
745cb0c
to
399eb12
Compare
399eb12
to
ac18b31
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, should wait for a sanity check from adam
sdk/js/src/Client.ts
Outdated
) | ||
} | ||
} | ||
if ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be an else if? otherwise it might open a new streaming connection and then immediately reconnect it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
true good point, fixed
sdk/js/src/StreamingConnection.ts
Outdated
@@ -11,6 +11,12 @@ export class StreamingConnection { | |||
this.openConnection() | |||
} | |||
|
|||
public updateURL(url: string): void { | |||
this.url = url |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might be better to change this property after you call close()
ac18b31
to
04d372d
Compare
04d372d
to
7519c00
Compare
7519c00
to
be7cfaf
Compare
No description provided.