Skip to content

cli: adjust clock adjustment to 30s#3819

Merged
roman-khimov merged 1 commit intomasterfrom
30s-token-nbf
Feb 13, 2026
Merged

cli: adjust clock adjustment to 30s#3819
roman-khimov merged 1 commit intomasterfrom
30s-token-nbf

Conversation

@roman-khimov
Copy link
Member

We have 20s max block time, this happens pretty often in practice:

Error: rpc error: deleting 1C1DCs3dkEuULKFTcnCFWh8X6dZvfLFVAXUgYe9EvfX object: remove object via client: status: code = 1028 message = malformed request: V2 token is invalid at 2026-02-13 08:56:53 +0000 UTC, token iat 2026-02-13 08:56:53 +0000 UTC, nbf 2026-02-13 08:57:03 +0000 UTC, exp 2026-02-13 18:57:03 +0000 UTC

We have 20s max block time, this happens pretty often in practice:

Error: rpc error: deleting 1C1DCs3dkEuULKFTcnCFWh8X6dZvfLFVAXUgYe9EvfX object: remove object via client: status: code = 1028 message = malformed request: V2 token is invalid at 2026-02-13 08:56:53 +0000 UTC, token iat 2026-02-13 08:56:53 +0000 UTC, nbf 2026-02-13 08:57:03 +0000 UTC, exp 2026-02-13 18:57:03 +0000 UTC
Signed-off-by: Roman Khimov <roman@nspcc.ru>
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

❌ Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 25.52%. Comparing base (0bac173) to head (584cb6e).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
cmd/neofs-cli/modules/object/util.go 0.00% 8 Missing ⚠️
cmd/neofs-cli/modules/session/create_v2.go 0.00% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3819      +/-   ##
==========================================
- Coverage   25.52%   25.52%   -0.01%     
==========================================
  Files         669      669              
  Lines       42931    42938       +7     
==========================================
  Hits        10959    10959              
- Misses      30967    30974       +7     
  Partials     1005     1005              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@roman-khimov roman-khimov merged commit 7835790 into master Feb 13, 2026
20 of 22 checks passed
@roman-khimov roman-khimov deleted the 30s-token-nbf branch February 13, 2026 11:40
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