Skip to content

Fix/sql schema exception handling#777

Open
odaysec wants to merge 1 commit intomicrosoft:Developmentfrom
odaysec:fix/sql-schema-exception-handling
Open

Fix/sql schema exception handling#777
odaysec wants to merge 1 commit intomicrosoft:Developmentfrom
odaysec:fix/sql-schema-exception-handling

Conversation

@odaysec
Copy link
Contributor

@odaysec odaysec commented Mar 6, 2026

Closes/supersedes #726

Changes:

  • Consolidated exception handling into single except block
  • Moved unreachable schema-building code back into try block
  • Sanitized user-facing error messages (removed str(e) to prevent info leakage)
  • Replaced print() with debug_print() for admin-controlled console output
  • Detailed exception info preserved in logs via log_event()

…l_schema_plugin

- Remove duplicate/unreachable except blocks
- Move schema-building logic back into try block
- Replace str(e) in user-facing response with generic message
- Use debug_print() instead of print() for admin-controlled output
- Keep detailed exception info in logs via log_event()
@odaysec odaysec changed the base branch from main to Development March 6, 2026 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant