Baseflow / flutter_cached_network_image

Download, cache and show images in a flutter app
https://baseflow.com
2.45k stars 656 forks source link

CORS errors on Flutter web #972

Open Kurczaak opened 2 months ago

Kurczaak commented 2 months ago

🐛 Bug Report

This bug happened to me after a while of using the package so it might not be easy to reproduce. I am unable to run my project due to CORS policies. This does not happen using Flutter's Image.network. I am fetching data from my Contentful CMS backend.

Expected behavior

Flutter web project run flawlesly

Reproduction steps

Run a Flutter Web app and try to load images from Contentful CMS backend.

Configuration

Version: 3.4.1

Platform:

Remy2701 commented 2 months ago

You can change the imageRenderMethodForWeb parameter to ImageRenderMethodForWeb.HttpGet instead of ImageRenderMethodForWeb.HtmlImage and that should fix your issue.

pauljonescodes commented 2 months ago

I'm still debugging this issue for my app so I'm not 100% sure on the specifics, but for what it's worth that fix did not work for me, and it appears the enum is a package other than this one I directly depend on.

Kurczaak commented 2 months ago

I'm currently not able to check it, but I recall I tried to go for other method but couldn't use the enum

pauljonescodes commented 2 months ago

I solved my problem by using the HTML renderer, it appears the default may have changed in some Flutter version upgrade. It would also appear like this is deprecated by Flutter, so I'm not sure I'd advise this solution for others if it can be avoided. Further, I'm accessing an S3 bucket which I do not control, I believe I could solve this is I controlled the CORS configuration and allowed all hosts.

rayliverified commented 2 months ago

Even with allow all hosts, images fail to load.

[
    {
        "origin": [
            "*"
        ],
        "method": [
            "GET"
        ],
        "maxAgeSeconds": 3600
    }
]

I wonder if it's because Wasm requires stricter headers. For example, deploying Wasm requires the server to serve these headers.

    {
      "source": "/(.*)",
      "headers": [
        {
          "key": "Cross-Origin-Embedder-Policy",
          "value": "credentialless"
        },
        {
          "key": "Cross-Origin-Opener-Policy",
          "value": "same-origin"
        }
      ]
    }

Wasm might require all files to be returned with

"responseHeader": ["Content-Type"],

max-anders commented 1 month ago

I encountered the following error: ImageCodecException: Failed to load network image.

Adding this helped resolve the issue: imageRenderMethodForWeb: ImageRenderMethodForWeb.HttpGet,

However, I am still facing another issue, GET https://lh3.googleusercontent.com/a/ACg8ocJEb--2bAckEYPPmMs8PL_tR031rtQ5pXJQF6ZNQWPQZBNnUw=s96-c 429 (Too Many Requests)

this is my cors

[ { "origin": ["*"], "method": ["GET"], "maxAgeSeconds": 3600 } ]

The images load fine on mobile, but I am encountering a 429 (Too Many Requests) error on the web.

luca1994ioma commented 1 month ago

I encountered the following error: ImageCodecException: Failed to load network image.

Adding this helped resolve the issue: imageRenderMethodForWeb: ImageRenderMethodForWeb.HttpGet,

However, I am still facing another issue, GET https://lh3.googleusercontent.com/a/ACg8ocJEb--2bAckEYPPmMs8PL_tR031rtQ5pXJQF6ZNQWPQZBNnUw=s96-c 429 (Too Many Requests)

this is my cors

[ { "origin": ["*"], "method": ["GET"], "maxAgeSeconds": 3600 } ]

The images load fine on mobile, but I am encountering a 429 (Too Many Requests) error on the web.

I have the same problem. Have you found a solution?

Remy2701 commented 1 month ago

For me changing the method worked. I did have to add the cached_network_image_platform_interface dependency. I'm guessing if you use the get method you'll probably have to allow the requests on your backend.

max-anders commented 4 weeks ago

@Remy2701

I have the same problem. Have you found a solution?

I've noticed that in my case only PNG images fails to load properly but everything seems to work fine in release mode for now

cedvdb commented 1 week ago

@rayliverified

Did you try what you suggested ?

TrSaka commented 1 week ago

Are you using firebase ?

15 Kas 2024 Cum 13:57 tarihinde cedvdb @.***> şunu yazdı:

@rayliverified https://github.com/rayliverified

Did you try what you suggested ?

— Reply to this email directly, view it on GitHub https://github.com/Baseflow/flutter_cached_network_image/issues/972#issuecomment-2478558090, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUTSNQ56BB46FPBBISYP3U32AXHR5AVCNFSM6AAAAABOQBGJ46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZYGU2TQMBZGA . You are receiving this because you are subscribed to this thread.Message ID: @.*** com>