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
fix 404 on blank.mp4 #1636
fix 404 on blank.mp4 #1636
Conversation
We definetly don't want to do that. |
I think we should decide to ship it, I'm fine with it. |
Conflicting files |
src/components/Videos.vue
Outdated
@@ -84,6 +84,8 @@ export default { | |||
options() { | |||
return { | |||
autoplay: this.active === true, | |||
// Used to reset the video streams https://github.com/sampotts/plyr#javascript-1 | |||
blankVideo: '../img/blank.mp4', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do that even works?
Shouldn't it be a geenrateImgPath or something? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
imagePath is the function name
5f55350
to
e30b34c
Compare
Signed-off-by: hamza221 <hamzamahjoubi221@gmail.com>
/backport to stable26 |
/backport to stable27 |
/backport to stable25 |
The blank.mp4 file is used to prevent MEDIA_ERR_SRC_NOT_SUPPORTED
The default value of blankVideo prop is cdn.plyr.io/static/blank.mp4. The previous value would result into 404 because /index.php/s/blank.mp4 doesn't exist, alternatively we can ship Nextcloud with a blank.mp4 file by default if we don't want to query an external cdn