Skip to content

skip redirector policy udpate in update_redirector_policy if proxy sever has not started yet. - #404

Merged
Zhidong Peng (ZhidongPeng) merged 3 commits into
Azure:devfrom
ZhidongPeng:msrc
Sep 9, 2026
Merged

skip redirector policy udpate in update_redirector_policy if proxy sever has not started yet.#404
Zhidong Peng (ZhidongPeng) merged 3 commits into
Azure:devfrom
ZhidongPeng:msrc

Conversation

@ZhidongPeng

Copy link
Copy Markdown
Collaborator

Context: GPA service redirects the azure host endpoints connections to localhost:3080, if the proxy_server could not bind to this local port. the redirected connections may reach random http://localhost:3080
Fix: check proxy server provisioning state, if it has not provisioned yet, skip update the redirector update policy. The redirector update will be happened only when Provision ALL_READY.

Test log with delayed proxy_server:

2026-09-08T20:33:35.593 [WARN]    Found key details from local and ready to use. - 631
2026-09-08T20:33:35.593 [WARN]    Proxy server is not provisioned, skipping redirector policy update.
2026-09-08T20:33:36.090 [INFO]    {"elapsed":1129, "message":"Started proxy listener, ready to accept request"}
2026-09-08T20:33:36.091 [WARN]    Started proxy listener, ready to accept request - 1129
2026-09-08T20:33:36.091 [INFO]    update_redirect_policy_internal with local ip address: 127.0.0.1, dest_ipv4: 168.63.129.16, dest_port: 80, local_port: 3080
2026-09-08T20:33:36.091 [INFO]    policy_map updated for destination: 168.63.129.16:80
2026-09-08T20:33:36.091 [INFO]    update_redirect_policy_internal with local ip address: 127.0.0.1, dest_ipv4: 169.254.169.254, dest_port: 80, local_port: 3080
2026-09-08T20:33:36.091 [INFO]    policy_map updated for destination: 169.254.169.254:80
2026-09-08T20:33:36.091 [INFO]    update_redirect_policy_internal with local ip address: 127.0.0.1, dest_ipv4: 168.63.129.16, dest_port: 32526, local_port: 3080
2026-09-08T20:33:36.091 [INFO]    policy_map updated for destination: 168.63.129.16:32526

@ZhidongPeng
Zhidong Peng (ZhidongPeng) merged commit d0a5b64 into Azure:dev Sep 9, 2026
24 checks passed
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