Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deleted note page which is indexed in search result throws an exception #3081

Open
hthabti opened this issue Mar 7, 2025 · 0 comments
Open
Labels

Comments

@hthabti
Copy link

hthabti commented Mar 7, 2025

issue description:
for an unknown reason, on customer instance(not reproduced locally), we have a note which is deleted on database, but still indexed in ES; what leads to a problem while filtering notes from tree drawer, when a specific note matches the typed keyword.

When this note is is retreived as a result and get itereated at this specific line

it generate an exception EntityNotFoundException which is catch outside the loop, here. Then the notes fetching is stopped before the end related line.

(ps: it is a swallowed error, without a correct LOG.error trace. We should add a line to log the error)

proposed solution: (discussed with Romain)
To make the code more robust, we should improve the error management, by adding a try catch around the for loop which will prevent the REST service to fail with error 500 even if a old note is still indexed

feedback:
77341

@hthabti hthabti added the Notes label Mar 7, 2025
@hthabti hthabti changed the title add try catch bloc to handle thrown exceprion while searching nots deleted not page which is indexed in serach result throws an exception and lead to the exit of the loop Mar 7, 2025
@rdenarie rdenarie changed the title deleted not page which is indexed in serach result throws an exception and lead to the exit of the loop deleted note page which is indexed in search result throws an exception Mar 7, 2025
@azayati azayati changed the title deleted note page which is indexed in search result throws an exception Deleted note page which is indexed in search result throws an exception Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

No branches or pull requests

1 participant