Skip to content

Feature implementation from commits 41b4ee6..80d649c#2

Open
yashuatla wants to merge 16 commits intofeature-base-2from
feature-head-2
Open

Feature implementation from commits 41b4ee6..80d649c#2
yashuatla wants to merge 16 commits intofeature-base-2from
feature-head-2

Conversation

@yashuatla
Copy link
Copy Markdown
Owner

@yashuatla yashuatla commented Jun 25, 2025

PR Summary

Refactor Context Keys to Use Typed Keys in Tests

Overview

This PR refactors the test code to use typed context keys instead of string literals, following Go best practices for context values.

Change Types

Type Description
Refactor Replace string literals with typed context keys in tests

Affected Modules

Module / File Change Description
entry_test.go Added contextKeyType type and replaced string literals with typed context keys

sirupsen and others added 16 commits June 13, 2022 07:17
update gopkg.in/yaml.v3 to v3.0.1
…-hooks-test-pkg

Fix data race in hooks.test package
This commit adds instructions to the syslog readme about how to
send different log levels to local logging (`log.SetLevel`) and
syslog hook.

fixes sirupsen#1369
…us.Writer() that could be triggered by logging text longer than 64kb without newlines. Previously, the bufio.Scanner used by Writer() would hang indefinitely when reading such text without newlines, causing the application to become unresponsive.
This commit fixes a potential denial of service
vulnerability in logrus.Writer() that could be
triggered by logging text longer than 64KB
without newlines. Previously, the bufio.Scanner
used by Writer() would hang indefinitely when
reading such text without newlines, causing the
application to become unresponsive.
This commit fixes a potential denial of service vulnerability in logrus.Writer() that could be triggered by logging text longer than 64kb without newlines.
Use text when shows the logrus output
This reverts commit 6acd903, reversing
changes made to e59b167.
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.

7 participants