Skip to content

fix: clock skew set to 1 min with correct format#237

Open
nitin-vavdiya wants to merge 1 commit into
SAP:mainfrom
nitin-vavdiya:fix/clock-skew
Open

fix: clock skew set to 1 min with correct format#237
nitin-vavdiya wants to merge 1 commit into
SAP:mainfrom
nitin-vavdiya:fix/clock-skew

Conversation

@nitin-vavdiya
Copy link
Copy Markdown

Description

Fixes the ClockSkew configuration value in appsettings.json to use the correct TimeSpan string format.

Why

Please include an explanation of why this change is necessary as well as relevant motivation and context. List any dependencies that are required for this change.

The current configuration sets ClockSkew as an integer:

"ClockSkew": 60000

When .NET binds a plain integer to a TimeSpan property, it interprets the value as days, not milliseconds. This results in a clock skew of approximately 164 years instead of the intended 1 minute.

Issue

236

Checklist

Please delete options that are not relevant.

  • I have followed the contributing guidelines
  • I have performed IP checks for added or updated 3rd party libraries
  • I have created and linked IP issues or requested their creation by a committer
  • I have performed a self-review of my own code
  • I have successfully tested my changes locally
  • I have added tests that prove my changes work
  • I have checked that new and existing tests pass locally with my changes
  • I have commented my code, particularly in hard-to-understand areas
  • I have added copyright and license headers, footers (for .md files) or files (for images)

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Dec 25, 2025

CLA assistant check
All committers have signed the CLA.

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.

1 participant