Skip to content

Cleanup set_connect_fail debug log#13091

Merged
masaori335 merged 3 commits intoapache:masterfrom
masaori335:asf-master-cleanup-set-connect-fail
Apr 21, 2026
Merged

Cleanup set_connect_fail debug log#13091
masaori335 merged 3 commits intoapache:masterfrom
masaori335:asf-master-cleanup-set-connect-fail

Conversation

@masaori335
Copy link
Copy Markdown
Contributor

Print name of errno to clarify.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates a debug log in HttpTransact::State::set_connect_fail() to print errno values in a more readable form (symbolic name/message) instead of raw integers.

Changes:

  • Replaces a Dbg() integer-based message with swoc::bwprint + swoc::bwf::Errno formatting.
  • Adds an explicit _dbg_ctl.on() guard before building the formatted message.

Comment thread include/proxy/http/HttpTransact.h Outdated
Comment thread include/proxy/http/HttpTransact.h
Comment thread include/proxy/http/HttpTransact.h
@masaori335 masaori335 merged commit 7e9aa23 into apache:master Apr 21, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this to For v10.2.0 in ATS v10.2.x Apr 21, 2026
@cmcfarlen cmcfarlen moved this from For v10.2.0 to Picked v10.2.0 in ATS v10.2.x Apr 23, 2026
@cmcfarlen cmcfarlen modified the milestones: 11.0.0, 10.2.0 Apr 23, 2026
@cmcfarlen
Copy link
Copy Markdown
Contributor

Cherry-picked to 10.2.x

cmcfarlen pushed a commit that referenced this pull request Apr 23, 2026
* Cleanup set_connect_fail debug log

* Fix args

* Include swoc header explicitly

(cherry picked from commit 7e9aa23)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Picked v10.2.0

Development

Successfully merging this pull request may close these issues.

4 participants