Skip to content

Commit de5c36f

Browse files
authored
chore: fix JavaScript lint errors (issue #9110)
## Description Fixed a `RangeError: Invalid time value` in `dispatch-workflow/lib/query.js` caused by calling `.toISOString()` on an invalid Date object when `info.reset` is undefined or not a valid number. ## Changes - Added `typeof` check to ensure `info.reset` is a number - Stored `getTime()` result in `time` variable - Used `isNaN()` guard before calling `.toISOString()` ## Related Issues resolves #9110 Signed-off-by: alok chando <chandoalok@gmail.com>
1 parent afe8812 commit de5c36f

1 file changed

Lines changed: 15 additions & 1 deletion

File tree

  • lib/node_modules/@stdlib/_tools/github/dispatch-workflow/lib

lib/node_modules/@stdlib/_tools/github/dispatch-workflow/lib/query.js

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,21 @@ function query( slug, id, options, clbk ) {
8383
info = ratelimit( response.headers );
8484
debug( 'Rate limit: %d', info.limit );
8585
debug( 'Rate limit remaining: %d', info.remaining );
86-
debug( 'Rate limit reset: %s', (new Date( info.reset*1000 )).toISOString() );
86+
87+
var resetDate;
88+
var time;
89+
90+
if ( typeof info.reset === 'number' ) {
91+
resetDate = new Date( info.reset * 1000 );
92+
time = resetDate.getTime();
93+
94+
debug(
95+
'Rate limit reset: %s',
96+
( isNaN( time ) ? 'invalid timestamp' : resetDate.toISOString() )
97+
);
98+
} else {
99+
debug( 'Rate limit reset: not provided' );
100+
}
87101

88102
if ( error ) {
89103
return clbk( error, info );

0 commit comments

Comments
 (0)