Skip to content

Vouch Request: nikhilgupta58 #3652

Closed as not planned
Closed as not planned

Description

@nikhilgupta58

What I want to work on

Fix for issue #3292 — OTLP nanosecond timestamp overflow in the webapp event repository.

Several functions in apps/webapp/app/v3/eventRepository/common.server.ts and apps/webapp/app/v3/runEngineHandlers.server.ts multiply epoch milliseconds by 1_000_000 before converting to BigInt, causing IEEE 754 precision loss (~256ns errors). The correct pattern (BigInt(ms) * BigInt(1_000_000)) already exists in convertDateToNanoseconds() in the same file.

The fix is 4 targeted lines across 2-3 files. I have already prepared the implementation and a .server-changes entry per CONTRIBUTING.md.

Background

I'm an active open source contributor. I recently contributed to calcom/cal.diy:

  • PR #29400: fix unknown rescheduleUid returning 200 instead of 404
  • PR #29401: fix OTP digit visibility regression in light-theme embed

Both PRs are under review. I follow the full contribution workflow: senior engineer review, CONTRIBUTING.md compliance, and CodeRabbit feedback addressed before marking ready for review.

Issue

Fixes #3292

Activity

  1. matt-aitken commented on Oct 7, 2026

    @matt-aitken
    Member

    We have a new Vouch policy, please see the discussion here: #5010

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions