BUG: Missing validation for empty or whitespace image_ids in Album requests

[Varun-JP]

Is there an existing issue for this?

• I have searched the existing issues

What happened?

Currently, the ImageIdsRequest schema allows empty lists or lists containing only whitespace strings (e.g., [" ", ""]). This can lead to database inconsistencies or unhandled 500 errors in the album management logic.

Proposed Fix: Add a Pydantic @field_validator to sanitize input and ensure the list contains valid, non-empty strings.

Record

• I agree to follow this project's Code of Conduct

[github-actions]

👋 Thanks for opening this issue! The maintainers will review it shortly. Till then, do not open any PR. This is a strict rule we like to follow around here :)

[Riddhi8077]

Hi @SIDDHANTCOOKIE , @AllMaintainers ,

This seems like a meaningful improvement for input validation, especially to prevent invalid or empty image_ids from reaching the database layer.

I’d be interested in working on this by adding validation (e.g., ensuring non-empty and trimmed strings using Pydantic). Before starting, I just wanted to confirm if this issue is open to contributions and whether there’s any preferred approach I should follow.

Thanks!

[SIDDHANTCOOKIE]

hey thanks for reviewing this issue and bringing up I'd let @Varun-JP to decide if he wants to work on it or not, and @rahulharpal1603 and @Zahnentferner will be the right people to ask for issue assignment!

[Riddhi8077]

Okay, thanks for letting me know!

[Varun-JP]

Hi @Riddhi8077 , thanks for your interest in this, I’ve already worked on this earlier and opened a PR, so I’ll continue with it for now. Feel free to take a look once it’s updated.

[Varun-JP]

Hi @SIDDHANTCOOKIE , since I’ve already worked on this earlier, could I be assigned to this issue so I can proceed with updating the PR?

[Varun-JP]

Hi @moderatoes @rahulharpal1603 @Zahnentferner, following up here , since I’ve already worked on this earlier and opened a PR, could I be assigned to this issue so I can proceed with updating it?

[Riddhi8077]

Hi @Riddhi8077 , thanks for your interest in this, I’ve already worked on this earlier and opened a PR, so I’ll continue with it for now. Feel free to take a look once it’s updated.

Yeah sure, No worries !