nick-boss-tech commented on PR #5110: URL: https://github.com/apache/solr/pull/5110#issuecomment-6392337175
**AI Revie: Three P2 findings:** - Silent remote failures: The new proxy helper drops failedNodes, so failed reads can appear as successful empty results. - Wrong error status: Its catch-all handler converts deliberate 400 errors for invalid/non-live nodes into 500s. - Permission-dependent navigation: Once cloud mode is known, a failed live-node lookup skips core-menu refresh. Users with core-read permission but without collection-admin-read can see stale core information. The earlier wrong-node mutation and session-restoration findings have fixes now. Maintainers also prefer #5112’s generic transport, keeping this approach as a fallback. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
