Skip to content

add support for transfers subscription#1578

Merged
stefangutica merged 10 commits intodevelopmentfrom
transfers-ws-subscription
Dec 17, 2025
Merged

add support for transfers subscription#1578
stefangutica merged 10 commits intodevelopmentfrom
transfers-ws-subscription

Conversation

@stefangutica
Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 15, 2025

k6 load testing comparison.
Base Commit Hash: 97255ec
Target Commit Hash: 2bd7476

Metric Base Target Diff
AvgMax9095AvgMax9095AvgMax9095
Nodes947.3920582.09118.668067.2715783.9319913.0919912.1819912.53+1566.04% 🔴-3.25% ✅+16680.93% 🔴+146.83% 🔴
Tokens58.851356.95100.82121.7354.20493.59100.35116.28-7.90% ✅-63.63% ✅-0.47% ✅-4.48% ✅
Accounts62.21471.3788.2395.5459.071173.8785.8692.81-5.06% ✅+149.04% 🔴-2.70% ✅-2.86% ✅
Blocks73.97456.53107.38126.4070.91826.29106.97122.93-4.14% ✅+80.99% 🔴-0.39% ✅-2.75% ✅
Pool55.371435.0282.8992.6251.84500.9681.7990.68-6.38% ✅-65.09% ✅-1.32% ✅-2.09% ✅
Mex55.341400.8882.9992.4152.241193.8782.1090.90-5.59% ✅-14.78% ✅-1.07% ✅-1.63% ✅
Transactions73.37277.45106.73129.1969.94292.27105.77122.71-4.68% ✅+5.34% 🔴-0.90% ✅-5.02% ✅
Test Run Duration60012.1060002.22

Legend: Avg - Average Response Time, Max - Maximum Response Time, 90 - 90th Percentile, 95 - 95th Percentile
All times are in milliseconds.

if (value !== undefined && value !== null && value !== '') {
activeFilters.push({ key, value });
if (key === 'token') {
if (data['value'] != null && data['value'] !== '' && data['value'] !== '0') {
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.

add data['value'] into const value = data['value']

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done

}
}

if (this.server.sockets.adapter.rooms.has(roomKey)) {
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.

duplicated code

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

solved

options,
);

const transfersFilteredForBroadcast: Map<string, Transaction[]> = new Map();
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.

add new line between variable declaration and for loop

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

added

Comment thread src/endpoints/websocket/entities/transfers.custom.payload.ts
Comment thread src/crons/websocket/room.key.generator.ts
Comment thread src/utils/mutual.exclusive.fields.constraint.ts Outdated
bogdan-rosianu
bogdan-rosianu previously approved these changes Dec 17, 2025
@stefangutica stefangutica merged commit f7c3c8d into development Dec 17, 2025
6 checks passed
@stefangutica stefangutica deleted the transfers-ws-subscription branch December 17, 2025 12:01
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