FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-82535: Ignore address resolution failure in SysLogHandler constructor by serhiy-storchaka · Pull Request #154504 · python/cpython · GitHub

Repository navigation

gh-82535: Ignore address resolution failure in SysLogHandler constructor - #154504

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:fix-syslog-lazy-connect
Aug 10, 2026
Merged

serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:fix-syslog-lazy-connect

Conversation

Copy link
Copy Markdown
Member

SysLogHandler resolves the address in the constructor, so creating a handler fails if the address is temporarily unresolvable, for example if DNS is not available yet.

Only the resolution failure is ignored, and the address is resolved again when a record is emitted. TimeoutError and ConnectionRefusedError are still raised from the constructor, so the behaviour added in gh-126400 is not affected.

🤖 Generated with Claude Code

…nstructor

The address may be temporarily unresolvable when the handler is created.
It is resolved again when a record is emitted.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
serhiy-storchaka added needs backport to 3.13 only security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes labels Aug 10, 2026
serhiy-storchaka merged commit c294654 into python:main Aug 10, 2026
62 checks passed
serhiy-storchaka deleted the fix-syslog-lazy-connect branch August 10, 2026 06:10

Copy link
Copy Markdown

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15.
🐍🍒⛏🤖

bedevere-app Bot commented Aug 10, 2026

Copy link
Copy Markdown

GH-155457 is a backport of this pull request to the 3.15 branch.

bedevere-app Bot removed the needs backport to 3.15 bugs and security fixes label Aug 10, 2026

bedevere-app Bot commented Aug 10, 2026

Copy link
Copy Markdown

GH-155458 is a backport of this pull request to the 3.14 branch.

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Aug 10, 2026

bedevere-app Bot commented Aug 10, 2026

Copy link
Copy Markdown

GH-155459 is a backport of this pull request to the 3.13 branch.

bedevere-app Bot removed the needs backport to 3.13 only security fixes label Aug 10, 2026
mbeijen pushed a commit to mbeijen/cpython that referenced this pull request Aug 14, 2026
…nstructor (pythonGH-154504)

The address may be temporarily unresolvable when the handler is created.
It is resolved again when a record is emitted.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Yhg1s pushed a commit that referenced this pull request Sep 29, 2026
…onstructor (GH-154504) (#155459)

gh-82535: Ignore address resolution failure in SysLogHandler constructor (GH-154504)

The address may be temporarily unresolvable when the handler is created.
It is resolved again when a record is emitted.
(cherry picked from commit c294654)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
serhiy-storchaka added a commit that referenced this pull request Oct 10, 2026
…onstructor (GH-154504) (GH-155457)

The address may be temporarily unresolvable when the handler is created.
It is resolved again when a record is emitted.
(cherry picked from commit c294654)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
serhiy-storchaka added a commit that referenced this pull request Oct 10, 2026
…onstructor (GH-154504) (GH-155458)

The address may be temporarily unresolvable when the handler is created.
It is resolved again when a record is emitted.
(cherry picked from commit c294654)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL