Skip to content

Conversation

@thrawn01
Copy link
Contributor

Purpose

Fix errcheck linter warnings that were causing CI failures after merging PR #21.

Implementation

  • Added error handling for r.ParseForm() in the /set HTTP handler
  • Added explicit ignore (_, _ =) for w.Write() calls in HTTP handlers where errors cannot be meaningfully handled
  • Wrapped deferred Close() calls in anonymous functions with explicit ignore to satisfy errcheck
  • Added proper error checking for sink.SetBytes() in tests
  • Changed dest.SetString() to return its error value in the getter function

thrawn01 and others added 30 commits June 13, 2023 09:16
Refuse further crosstalk from crosstalk requests.
On mem full, evict all expired keys.
udhos and others added 29 commits March 14, 2025 01:19
Send two optional context fields to getter thru protobuf.
Fix optional user-supplied per-request context information.
Address errcheck linter warnings by properly handling error return values
from functions like Write, Close, ParseForm, and others.
@thrawn01 thrawn01 closed this Dec 25, 2025
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.

3 participants