CODEWITHSUNDEEP
phpsymfonystreamheaderresponse
A.Bmt
A.Bmt

Reputation: 23

Symfony 2.7 set header from StreamedResponse's callback

I have to set dynamically my Content-Type into the StreamedResponse's callback.

I wrote a code like this that it works fine on Symfony 2.3 but not on 2.7. The returned Content-Type is text/html.

function indexAction()
{
    $r = new \Symfony\Component\HttpFoundation\StreamedResponse();
    $r->setCallback(function() use ( & $r)
    {
        // ...

        $r->headers->set('Content-Type', 'application/json');

        // ...
    });
    return $r;
}

I found this commit that it can be on but i don't really understand... https://github.com/symfony/http-foundation/commit/6a0838a26d54eff153b825e1550c1f6fa05a0941

It looks like we can now only set header outside the callback.

It works if i send headers with the native php function into the closure but it's dirty...

header('Content-Type: application/json');

if someone has a clue...

Thanks!

Upvotes: 2

Views: 1532

Answers (2)

Nikita U.
Nikita U.

Reputation: 3618

To be accurate, you can use hack with replacing headers up to version 2.7.19, it was merged in 2.7.20

As @MichaelSivolobov said in his answer, I also have never seen cases when you need it

I'm sure such commits are always made for enforce better application design. When I see such cases as in your example, it's a some kind of red flag to rethink code/data flow

Upvotes: 1

Michael Sivolobov
Michael Sivolobov

Reputation: 13240

Streamed response made for purposes when you need long-polling connections. And so you have to know what content-type it will serve before you send any data.

When you return your Response object in action Symfony already sends headers and starts to stream output from callback function. In this moment it is not right to send another header.

P.S. I have never seen case when you really need StreamedResponse and need to set Content-Type dynamically.

Upvotes: 2

Related Questions