Fix api-tokens date display + quiet authIO no-token log (#36)
- api_tokens.ejs: created_on/last_used_on come back from Redis as strings (model-redis only coerces fields with an explicit `type`), so `new Date(ms)` yielded "Invalid date". Use `moment(ms, "x")` (the hosts.ejs/dns.ejs precedent) which parses a numeric string-or-number as a Unix-ms timestamp. - api_tokens.ejs: `isExpired` is a class getter not serialized to the client JSON, so the "expired" badge never showed — compute expiry in the view via `Date.now() > Number(expires_at)`. Also guard the `last_used_on: 0` / falsy case (string "0" is truthy) so unset timestamps render "—" not "1970". - middleware/auth.js: authIO did `checkToken(socket.handshake.auth.token || 0)`, so any socket connect without a token (login page, pre-login) did an `AuthToken.get(0)` lookup and logged a noisy `EntryNotFound` trace. Guard: reject the socket with a generic 401 when there's no token (behavior- preserving — unauth sockets were already rejected; just no Redis lookup / 404). Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
@@ -80,13 +80,21 @@
|
||||
}
|
||||
|
||||
function fmtTime(ms){
|
||||
if(!ms) return '—';
|
||||
return moment(new Date(ms)).fromNow() + ' <span class="text-muted">(' + moment(new Date(ms)).format('YYYY-MM-DD HH:mm') + ')</span>';
|
||||
// created_on/last_used_on come back from Redis as strings (model-redis
|
||||
// only coerces fields with an explicit `type`); moment(value, "x") parses
|
||||
// a numeric string-or-number as a Unix-ms timestamp, unlike new Date(str).
|
||||
if(!ms || Number(ms) === 0) return '—';
|
||||
var t = moment(ms, "x");
|
||||
if(!t.isValid()) return '—';
|
||||
return t.fromNow() + ' <span class="text-muted">(' + t.format('YYYY-MM-DD HH:mm') + ')</span>';
|
||||
}
|
||||
function fmtExpiry(token){
|
||||
if(!token.expires_at) return '<span class="badge bg-secondary">never</span>';
|
||||
if(token.isExpired) return '<span class="badge bg-danger">expired</span>';
|
||||
return '<span class="badge bg-warning text-dark">' + moment(new Date(token.expires_at)).fromNow() + '</span>';
|
||||
// expires_at is type:number (a real number); isExpired is a class getter
|
||||
// that is NOT serialized to the client, so compute expiry here.
|
||||
var exp = Number(token.expires_at);
|
||||
if(!exp) return '<span class="badge bg-secondary">never</span>';
|
||||
if(Date.now() > exp) return '<span class="badge bg-danger">expired</span>';
|
||||
return '<span class="badge bg-warning text-dark">' + moment(exp, "x").fromNow() + '</span>';
|
||||
}
|
||||
|
||||
function processToken(token){
|
||||
|
||||
Reference in New Issue
Block a user