markjoin123 opened a new pull request, #7861:
URL: https://github.com/apache/shenyu/pull/7861

   Fixes #7732
   
   Gateway instance identity now uses the port the application server actually 
bound, including when `server.port=0`. The separate management listener is 
ignored, and nothing is published until a positive application port is observed.
   
   - Independent heartbeat keeps its single scheduler and skips reports until 
`WebServerInitializedEvent` supplies a positive application port. Shutdown 
still cancels that scheduler, and a later initialization event does not start 
another one.
   - Websocket sync defers client creation until that port is known, so the 
`ClientPort` handshake header is the bound port.
   - HTTP long polling omits `X-Real-PORT` until the bound port is known. Admin 
ignores a missing, zero, or non-numeric client port instead of publishing an 
empty identity.
   
   Make sure that:
   
   - [x] You have read the [contribution 
guidelines](https://shenyu.apache.org/community/contributor-guide).
   - [x] You submit test cases (unit or integration tests) that back your 
changes.
   - [ ] Your local test passed `./mvnw clean install 
-Dmaven.javadoc.skip=true`.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to