Skip to content

Conversation

@Akvicor
Copy link

@Akvicor Akvicor commented Aug 3, 2025

@tangyongfu
Copy link

tangyongfu commented Aug 3, 2025 via email

@Akvicor Akvicor changed the title fix tcp check on 1.28.0 && support https check && support prometheus … Nginx1.28.patch && fix tcp check && support https check && support prometheus Aug 3, 2025
+ ngx_log_debug1(NGX_LOG_DEBUG_HTTP, pc->log, 0,
+ "get hash peer, check_index: %ui", peer->check_index);
+ if (ngx_http_upstream_check_peer_down(peer->check_index)) {
+ goto next;

Choose a reason for hiding this comment

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

missing ngx_http_upstream_rr_peer_unlock(hp->rrp.peers, peer);?

@tangyongfu
Copy link

tangyongfu commented Dec 10, 2025 via email

+ "get ip_hash peer, check_index: %ui",
+ peer->check_index);
+ if (ngx_http_upstream_check_peer_down(peer->check_index)) {
+ goto next;

Choose a reason for hiding this comment

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

missing ngx_http_upstream_rr_peer_unlock(iphp->rrp.peers, peer); ?

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