fix: use zero-length slice with capacity for EmailTemplates by lakhansamani · Pull Request #496 · authorizerdev/authorizer

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@lakhansamani

Copy link Copy Markdown

Contributor

Summary

  • Changed make([]*model.EmailTemplate, len(emailTemplates)) to make([]*model.EmailTemplate, 0, len(emailTemplates))
  • Eliminates nil-padded entries in the API response

Test plan

  • Verify EmailTemplates query returns correct data without nil entries

Fixes #485

The slice was created with len(emailTemplates) nil elements, then
append added more entries after the nil padding. Changed to 0 length
with pre-allocated capacity.

Fixes #485

@lakhansamani lakhansamani merged commit e38d88d into main

Mar 1, 2026

@lakhansamani lakhansamani deleted the fix/email-templates-nil-slice branch

March 1, 2026 07:07

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

EmailTemplates returns nil-padded slice

1 participant

@lakhansamani