Skip to content

fix: drop server-side remnants, add WS interface - #1819

Open
arnautov-anton wants to merge 4 commits into
release-v10from
drop-server-side-remnants
Open

fix: drop server-side remnants, add WS interface#1819
arnautov-anton wants to merge 4 commits into
release-v10from
drop-server-side-remnants

Conversation

@arnautov-anton

Copy link
Copy Markdown
Contributor

Description of the changes, What, Why and How?

@arnautov-anton arnautov-anton changed the title Initial commit fix: drop server-side remnants, add WS interface Aug 11, 2026
@arnautov-anton
arnautov-anton force-pushed the drop-server-side-remnants branch from 1b48929 to 2790716 Compare August 11, 2026 16:28
Comment thread src/utils.ts Outdated
Comment on lines 103 to 113
export function addFileToFormData(uri: string | Blob, name?: string) {
const data = new FormData();

if (isReadableStream(uri) || isBuffer(uri) || isFileWebAPI(uri) || isBlobWebAPI(uri)) {
if (isBlobWebAPI(uri)) {
if (name) data.append('file', uri, name);
else data.append('file', uri);
} else {
data.append('file', {
uri,
name: name || (uri as string).split('/').reverse()[0],
contentType: contentType || undefined,
type: contentType || undefined,
});
data.append('file', uri);
}

return data;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This right here is the biggest change and would need some testing from RN guys too, not sure whether this is the correct typing. @isekovanic

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checking

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in dd95783

Comment thread src/connection.ts
Comment on lines +345 to +346
const WS = this.client.options.WebSocketImpl ?? WebSocket;
this.ws = new WS(wsURL);

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread src/signing.ts
* @returns The uncompressed payload bytes.
* @throws {@link InvalidWebhookError} when the gzip envelope is malformed.
*/
export function gunzipPayload(rawBody: string | Buffer): Buffer {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@szuperaz do we have these APIs in the node-sdk?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, they were added to stream-node too when they were added to stream-chat-js: GetStream/stream-node#294

@arnautov-anton
arnautov-anton force-pushed the drop-server-side-remnants branch from 2790716 to 138ddc9 Compare August 12, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants