Aias00 opened a new issue, #6547:
URL: https://github.com/apache/shenyu/issues/6547

   ## Description
   In the `catch` of `existsIndex` the method returns `true`. `consume0` then 
treats the index as existing and skips `createIndex`/alias setup. On any 
transient error checking index existence (network blip, auth 401, ES 
temporarily unreachable) while the daily index does not yet exist, the day's 
index is never created; the subsequent `client.bulk(...)` then fails (if 
`auto_create_index` is disabled, common in secured clusters), so the **entire 
batch is dropped** on every failed check until the check happens to succeed.
   
   ## Location
   ```
   
shenyu-plugin-logging-elasticsearch/.../client/ElasticSearchLogCollectClient.java:134-142
 (used at consume0 107-110)
   ```
   
   ## Impact
   All log batches throw on insert; `AbstractLogCollector` catches and drops 
them, so no logs are persisted until the check happens to succeed.
   
   ## Suggested fix
   Return `false` on error (fail-safe: attempt creation) or rethrow so the 
batch is retried; do not claim the index exists when the check itself failed.
   
   ## Related existing issue(s)
   None
   
   _Identified during the 2026-08-02 audit; full list in 
[`docs/issue-candidates-2026-08-02.md`](docs/issue-candidates-2026-08-02.md)._


-- 
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