Skip to content

fix(utils): remove tx.Debug() from PerformLicenseMapActions#208

Open
aaryan359 wants to merge 1 commit intofossology:mainfrom
aaryan359:fix/remove-debug-sql-logging
Open

fix(utils): remove tx.Debug() from PerformLicenseMapActions#208
aaryan359 wants to merge 1 commit intofossology:mainfrom
aaryan359:fix/remove-debug-sql-logging

Conversation

@aaryan359
Copy link
Copy Markdown
Contributor

Changes

  • Removes hardcoded tx.Debug() call from PerformLicenseMapActions in pkg/utils/util.go.
  • .Debug() forced GORM to print every SQL query and its arguments (including UUIDs) to stdout regardless of the configured log level, causing sensitive data exposure and unnecessary performance overhead in production.

Submitter Checklist

  • Includes tests (if there is a feature changed/added)
  • Includes docs ( if changes are user facing)
  • I have tested my changes locally.

Signed-off-by: aaryan359 <aaryanmeena96@gmail.com>
@aaryan359
Copy link
Copy Markdown
Contributor Author

@deo002,
Found a tx.Debug() left in PerformLicenseMapActions — it was leaking all SQL queries
and UUID args to stdout in production, which is both a security risk and a performance hit.

Single-word fix — removed .Debug().

Hope this helps!

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