v1/internal: refactor GetComposeStatus improve error handling

[mgold1234]

this commit improve error handling and code structure

• Split GetComposeStatus into smaller, more focused functions
• Enhance error handling with more specific HTTP status codes *Add dedicated functions for handling different response scenarios
• Implement parseAndRedactComposeRequest for better separation of concerns
• Improve code readability and maintainability

[codecov]

Codecov Report

Attention: Patch coverage is 65.71429% with 12 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
internal/v1/handler.go	65.71%	7 Missing and 5 partials ⚠️

Files with missing lines	Coverage Δ
internal/v1/handler.go	69.28% <65.71%> (+0.37%)	⬆️

[schuellerf]

While this makes the functions smaller and more readable
it duplicates the number of requests (calling h.getComposeByIdAndOrgId(ctx, composeId) twice now)?
If I'm right - this should be optimized to avoid duplicating the number of external requests

[mvo5]

Thank you, I'm not familiar with the image-builder codebase so please take my review with a grain of salt. I have some suggestions inline about structure and naming though that are maybe/hopefully helpful :)

[mvo5]

Thank you and sorry that it took me so long to look at this! This looks nice, some tiny naming/code organisation suggestions inline for your consideration.