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.

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

Return the total amount of data volume used #608

Closed
wants to merge 1 commit into from

Conversation

berrnd
Copy link

@berrnd berrnd commented Apr 3, 2024

Hi and first thanks a lot for LibreSpeed!

I wanted to show how much data volume the upload / download test used. So I've added 2 properties to the speedtest_worker.js status data object:

  • dlBytes: Total number of bytes downloaded
  • ulBytes: Total number of bytes uploaded

This also includes data during dlGraceTime / ulGraceTime.

Could look like that (a live example is available here):

firefox_2024-04-03_20-11-11.mp4

@sstidl sstidl deleted the branch librespeed:dev May 18, 2024 19:51
@sstidl sstidl closed this May 18, 2024
@berrnd

This comment was marked as resolved.

@berrnd berrnd deleted the add-used-data-volume branch May 18, 2024 20:35
@sstidl
Copy link
Collaborator

sstidl commented May 24, 2024

@berrnd im sorry, this was not intentional. i deleted the dev branch as it was confusing, this had the consequence that it deleted/closed all PRs in this branch.

@sstidl
Copy link
Collaborator

sstidl commented May 24, 2024

@berrnd could you please resubmit if you have the time? i tried to recover the changes but you had already deleted your fork

@berrnd
Copy link
Author

berrnd commented May 24, 2024

i deleted the dev branch as it was confusing, this had the consequence that it deleted/closed all PRs in this branch.

Didn't know that, without any further hint in the event timeline here this IMHO isn't very well thought out from GitHub - then sorry for my reaction, it just looked like that to me at first.

 

could you please resubmit if you have the time?

Restored my fork and resubmitted this PR: => #627

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.

None yet

2 participants