The name of the file being downloaded is “download” without extension about laravel-zipstream

[jszobody]

I can't reproduce. That code you pasted just gave me a OpI4O73yLqV92K4I5PRU.zip download file.

Is that download method in your Controller? Not sure where $this->files is coming from, but I'm not thinking this is a controller method.

Something else is getting in the way, and I would need to see the controller itself to help you further.

[lucdaalmeijer]

What was the solution for this? I'm having the same issue, This is my code:

public function downloadDocumentsV2(int $year, string $type, Collection $documents = null)
{
    if (is_null($documents)) {
        $documents = $this->getDocumentsForTypeGroupWithYear($year, $type)->get();
    }

    $zip = Zip::create('package.zip');

    foreach ($documents as $document) {
        $document = $document->files->first();
        $zip->add("s3://". config('filesystems.disks.s3.bucket') . '/' . $document->path, $document->name);
    }

    return $zip;
}

@ingangelbaeza

[lucdaalmeijer]

or @jszobody if you have a solution for this 🙂

[jszobody]

@lucdaalmeijer Is downloadDocumentsV2 a controller method? Any chance it's being returned instead from a Livewire component?

[lucdaalmeijer]

yes downloadDocumentsV2 is a controller method and is being returned from an livewire component like this:

return $this->entity->downloadDocumentsV2($this->year, $this->type);

[jszobody]

That's your issue, you can't return a zip stream from a LiveWire component. LW wants to grab the entire response payload and serialize it inside its own JSON response, you lose all the benefits of streaming out the response (it doesn't start instantly) and LW gets in the way of the filename header.

More explanation:
#34 (comment)

Alternate approach:
#34 (comment)

[lucdaalmeijer]

ohw i will look into that, thank you for the insight, response and the alternate approach!