Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Video content types not working since 1.3.1 #293

Closed
iainbeeston opened this issue Nov 13, 2024 · 7 comments · Fixed by #296
Closed

Video content types not working since 1.3.1 #293

iainbeeston opened this issue Nov 13, 2024 · 7 comments · Fixed by #296
Assignees
Labels
bug Something isn't working

Comments

@iainbeeston
Copy link
Contributor

Since 1.3.1 I've had issues with validating the content type of videos (I originally mentioned this in #280 but that issue has been closed). I haven't been able to tell what is causing the issue but I have been able to reproduce in a test: cf953b31fbb5f053234852fe2b827d767abed80c. The test passes on 1.3.0 and earlier, but fails on 1.3.1+ (including 1.3.3) with the validation error: "Video has an invalid content type"

@iainbeeston iainbeeston changed the title Video attachments not working since 1.3.1 Video content types not working since 1.3.1 Nov 13, 2024
@iainbeeston
Copy link
Contributor Author

I assume this must be related to #292 and #282

@Mth0158
Copy link
Collaborator

Mth0158 commented Nov 13, 2024

Hi @iainbeeston,
Could you provide a bit more context? Like the model validations + the way you are attaching the file? It'll help solve the issue

@Mth0158 Mth0158 added the bug Something isn't working label Nov 13, 2024
@Mth0158 Mth0158 self-assigned this Nov 14, 2024
Mth0158 added a commit that referenced this issue Nov 14, 2024
…ot-working-since-131

[Validator] Remove extension check on content_type validator (#282/#291/#292/#293)
@Mth0158 Mth0158 reopened this Nov 14, 2024
@Mth0158
Copy link
Collaborator

Mth0158 commented Nov 14, 2024

Incoming bugfix release in the next hours/tomorrow to definitely fix this issue => we do not perform the extension vs content_type that causes this mess. Sorry for the unexpected breaking changes, it should have been tested more.

Let us know if you have any features idea to enhance the gem by the way :)

@iainbeeston
Copy link
Contributor Author

@Mth0158 Great, thanks. Could you include the test from master...iainbeeston:active_storage_validations:validate-video-attachments in your fix? That reproduces the issue I'm seeing and should help prevent it happening again

@Mth0158
Copy link
Collaborator

Mth0158 commented Nov 14, 2024

Of course, here it is, #297, it will be released in the same fix

@Mth0158
Copy link
Collaborator

Mth0158 commented Nov 14, 2024

The fix has been released, upgrade to 1.3.4 to fix the issue :)

@Mth0158 Mth0158 closed this as completed Nov 14, 2024
@iainbeeston
Copy link
Contributor Author

@Mth0158 It works for me! Thank you so much for all the work over the last few days to fix bugs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants