Asp.net Exception of type 'System.OutOfMemoryException

You're basically just eating up too much memory as the entire file contents of 500 are stored in your memory stream.
You can get around it with a bit of a rework.  Solution is to use the Response.OutputStream directly as the target of the ZipArchive.  The hiccup, is that the .net implementation "thinks" that it needs to be a seekable stream, and returns an error if you try to do that.

But it doesn't actually have to be seekable.  If you make a stream wrapper class that overrides the "CanSeek" property, and use that to pass the outputstream directly to the ziparchive you don't consume any memory, and you don't have the lag of creating the zip in the first place.

Apologies, I have the class in c# not vb, but I'm sure you can pick up the critical parts.

    public class ZipStreamWrapper : Stream
    {
        public ZipStreamWrapper(Stream stream)
        {
            baseStream = stream;
            length = 0;
        }

        private int length;
        private Stream baseStream;

        public override bool CanRead { get { return true; } }
        public override bool CanSeek { get { return false; } }
        public override bool CanWrite { get { return true; } }

        public override long Length
        {
            get { return length; }
        }

        public override long Position
        {
            get { return length; }
            set
            {
                throw new NotImplementedException();
            }
        }

        public override void Flush()
        {
            baseStream.Flush();
        }
        public override int Read(byte[] buffer, int offset, int count)
        {
            return baseStream.Read(buffer, offset, count);
        }
        public override long Seek(long offset, SeekOrigin origin)
        {
            throw new NotImplementedException();
        }
        public override void SetLength(long value)
        {
            throw new NotImplementedException();
        }
        public override void Write(byte[] buffer, int offset, int count)
        {
            baseStream.Write(buffer, offset, count);
            length += count;
        }
    }

Select allOpen in new window

And then just create the archive with this instead:

new ZipArchive(new ZipStreamWrapper(Response.OutputStream), ZipArchiveMode.Create))

Select allOpen in new window

You're using a contiguous region of memory to load the stream, so on large files you're pretty much always going to get that exception.  You'll want to use some sort of FileStream.

What may be helpful is to take a look at this blog post: https://blog.stephencleary.com/2016/11/streaming-zip-on-aspnet-core.html

Writing directly to the output stream will be far better on resources, speed and latency than using an intermediate filestream.
Try the stream wrapper as suggested.

Hi guys,
Thank you for your advises.
I ended up using filestream  which solve my problem thank you all.