Skip to content

Improve error handling in upsert#14432

Merged
mattiagiupponi merged 2 commits into
masterfrom
improved-upsert-error-handling
Jul 15, 2026
Merged

Improve error handling in upsert#14432
mattiagiupponi merged 2 commits into
masterfrom
improved-upsert-error-handling

Conversation

@giohappy

@giohappy giohappy commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@giohappy giohappy marked this pull request as ready for review July 13, 2026 09:42
@giohappy giohappy changed the title Potential fix for code scanning alert no. 3578: Information exposure through an exception Improve error handling in upsert Jul 13, 2026
@giohappy giohappy requested a review from mattiagiupponi July 13, 2026 09:43

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors exception handling in the vector upload handler to log the detailed traceback and return a generic, user-friendly error message. The review feedback is highly constructive, pointing out opportunities for internationalization (i18n), resolving potential dead code, and preserving the original exception context using exception chaining. No comments need to be removed.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread geonode/upload/handlers/common/vector.py
@cla-bot cla-bot Bot added the cla-signed CLA Bot: community license agreement signed label Jul 13, 2026
@mattiagiupponi mattiagiupponi merged commit 8c6a987 into master Jul 15, 2026
16 checks passed
@mattiagiupponi mattiagiupponi deleted the improved-upsert-error-handling branch July 15, 2026 07:58
mattiagiupponi pushed a commit that referenced this pull request Jul 15, 2026
* Log to err upsert exception



* fixed msg formatting

---------


(cherry picked from commit 8c6a987)

Co-authored-by: Giovanni Allegri <giohappy@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 5.1.x cla-signed CLA Bot: community license agreement signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants