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.

Sign up for GitHub

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

Jump to bottom

undefined in getstats #87

Closed
fippo opened this issue Jun 8, 2016 · 2 comments
Closed

undefined in getstats #87

fippo opened this issue Jun 8, 2016 · 2 comments

Comments

@fippo
Copy link
Owner

fippo commented Jun 8, 2016 

               "undefined" : "2016-06-03T20:45:52.297Z",

This oddly looks as if the timestamp is set wrong
@fippo
Copy link
Owner Author

fippo commented Jun 8, 2016

possibly from https://github.com/opentok/rtcstats/blob/master/rtcstats.js#L312

@fippo fippo closed this as completed Feb 21, 2019
@fippo fippo reopened this Feb 21, 2019
@fippo
Copy link
Owner Author

fippo commented Feb 21, 2019

still an issue. For entries with type === 'getStats' this has the datetime set but the key is undefined. And having the time on the entry itself would be useful.

fippo added a commit that referenced this issue Feb 21, 2019
@fippo
dont mangle timestamp on stats
ac94802 
fixes #87
@fippo fippo mentioned this issue Feb 21, 2019
Merged
fippo added a commit that referenced this issue Feb 21, 2019
@fippo
dont mangle timestamp on stats
28630e5 
fixes #87
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fippo