Setting Content_Type

[shortmort37]

Same error (just a line further down with the "use" added).

Catchable fatal error: Argument 1 passed to Symfony\Component\HttpFoundation\BinaryFileResponse::prepare() must be an instance of Symfony\Component\HttpFoundation\Request, instance of phpbb\request\request given, called in /home/sportf5/public_html/59test/includes/functions_download.php on line 143 and defined in /home/sportf5/public_html/59test/vendor/symfony/http-foundation/Symfony/Component/HttpFoundation/BinaryFileResponse.php on line 169

[kasimi]

Post the whole if ($category == ATTACHMENT_CATEGORY_HTML5 ...) block.

[shortmort37]

These are my mods - at the top,

Code: Select all

use Symfony\Component\HttpFoundation\BinaryFileResponse;
use phpbb\symfony_request;

Then:

Code: Select all

	if (!@file_exists($filename))
	{
		send_status_line(404, 'Not Found');
		trigger_error('ERROR_NO_ATTACHMENT');
	}
	// *** Adding HTML5 audio/video content!  Caveat Emptor
	if (($category == ATTACHMENT_CATEGORY_HTML5)
		&& ((strpos($attachment['mimetype'], 'video') !== 0)||(strpos($attachment['mimetype'], 'audio') !== 0)))
	{
		$response = new BinaryFileResponse($filename);
		$response->headers->set('Content-Type', 'video/mp4');
		global $request;
		$response->prepare($request); 
		$response->send();
		exit;
	}
	// Correct the mime type - we force application/octetstream for all files, except images
	// Please do not change this, it is a security precaution
	if (($category != ATTACHMENT_CATEGORY_IMAGE || strpos($attachment['mimetype'], 'image') !== 0))
...

I've forced the Content-Type here to be video, but of course if I get it to work I would set it to audio/video as appropriate.

[kasimi]

You missed the last line in this post.

[shortmort37]

Indeed I did! It reminds me of a joke:

Q: What does a dyslexic agnostic insomniac do?
A: He lies awake all night, and wonders if there really is a dog.

(!)

Hey, it works!! Not only that...




It works on my iPad, which was the whole objective of this little exercise!

Many, many thanks. I will make revision to add mp3 capability as well. This is awesome!

Dan

[playagiron]

Hi,

I know it is an old topic.

But struggling with the same issues of not playing MP3 files in the latest 3.3.11 version.

Bit hard to follow how to finally achieve the right "audio/mpeg" mimetype for mp3 attachments.

Anyone still following who could give the whole set of mods required?

Thanks a dozen!
Johannes

[Mick]

phpBB no longer uses or recommends mods as extensions have been in use since 3.1. I suggest you make an extension request, you can link back to this topic if you wish.

Are you saying if you attach an mp3 file you can’t play it? I’ve just tried it and it works fine:

IMG_0424.jpeg

Are you sure you have the attachment settings set correctly?

[playagiron]

Hi thanks! yes I see the differences from MODs to EXT 8just migrated my board).

It DOES work on PC, linux whatnot. Just NOT on iOS devices, either browser,
as the emimetype for non-images is always octet-stream. And not audio/mpeg.
hard coded files work but the download PHP links get this general mimetype which Apple seems not to recognize as playable html5 files.

[playagiron]

This part in the code includes/functions_download.php

Code: Select all

	// Correct the mime type - we force application/octetstream for all files, except images
	// Please do not change this, it is a security precaution
	if ($category != ATTACHMENT_CATEGORY_IMAGE || strpos($attachment['mimetype'], 'image') !== 0)
	{
		$attachment['mimetype'] = (strpos(strtolower($user->browser), 'msie') !== false || strpos(strtolower($user->browser), 'opera') !== false) ? 'application/octetstream' : 'application/octet-stream';
	}
	

seems to set the mime-type. I tried also adding an exception for audio files to test if it works but seems to not change anything. Any other place I need to change to test passing the mime type from the SQL attachment tables instead of octetstream?

[Paul]

This forum is for extensions writers support. If you have a issue with phpBB features itself, you should post in the support forum and provide the SRT so people can give proper support.

[Mick]

It DOES work on PC, linux whatnot. Just NOT on iOS devices

The example I posted was on a MacBook with Safari and FF so I’m not sure what the issue is with iOS, I tried iOS as well and it wanted to download the file which I did and it played ok.

This may be useful https://discussions.apple.com/thread/25 ... ortBy=best

[playagiron]

This forum is for extensions writers support. If you have a issue with phpBB features itself, you should post in the support forum and provide the SRT so people can give proper support.

Hi Paul,

yes I noted. And had in fact created a SRT post here: viewtopic.php?p=16011094#p16011094
But was forwarded to this topic, which is not fully complete at least it seems (in the sense I cannot replicate all the modifications required to make it work on my board.
In case someone has another idea or summary of all the changes that are required to pass the right mime type for files here mp3s would be much appreciated! Thanks!