Skip to content

fix(status): print SSE log timestamps with correct integer width#519

Merged
stackia merged 1 commit into
mainfrom
codex/fix-status-log-timestamps
Jun 12, 2026
Merged

fix(status): print SSE log timestamps with correct integer width#519
stackia merged 1 commit into
mainfrom
codex/fix-status-log-timestamps

Conversation

@stackia

@stackia stackia commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix SSE log JSON serialization to format timestamp with a 64-bit integer specifier.
  • Prevents truncation or undefined output on platforms where time_t is wider than long.

Testing

  • Not run (not requested)

@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://thankful-water-0a297bf00-519.eastasia.1.azurestaticapps.net

@stackia stackia force-pushed the codex/fix-status-log-timestamps branch from f6e057d to 787d79a Compare June 12, 2026 03:36
@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://thankful-water-0a297bf00-519.eastasia.1.azurestaticapps.net

Comment thread src/status.c
(long)status_shared->log_entries[log_idx].timestamp, status_shared->log_entries[log_idx].level,
status_get_log_level_name(status_shared->log_entries[log_idx].level), escaped);
len +=
snprintf(buffer + len, buffer_capacity - (size_t)len,
@stackia stackia force-pushed the codex/fix-status-log-timestamps branch from 787d79a to b806bba Compare June 12, 2026 03:40
@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://thankful-water-0a297bf00-519.eastasia.1.azurestaticapps.net

@stackia stackia force-pushed the codex/fix-status-log-timestamps branch from b806bba to cb16810 Compare June 12, 2026 03:42
@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://thankful-water-0a297bf00-519.eastasia.1.azurestaticapps.net

@stackia stackia marked this pull request as ready for review June 12, 2026 03:43
@stackia stackia merged commit 305b299 into main Jun 12, 2026
9 of 10 checks passed
@stackia stackia deleted the codex/fix-status-log-timestamps branch June 12, 2026 03:44
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.

2 participants