rancher/partner-charts

Dxemssql 500 Icon Get Response

nicholasSUSE opened this issue · 5 comments

Hello, the icon is getting a 500 server response, we have tested it with different header possibilities, and it worked with an unusual configuration like (User-Agent: "").

We have decided not to implement the workaround, the chart owners should fix this unusual behavior on their side.

Task that generated this investigation: 404s at the Chart Marketplace

cURL request for making debugging easier, you will have to replace the cookies with yours:
curl 'https://localhost:8443/v1/catalog.cattle.io.clusterrepos/rancher-partner-charts?chartName=dxemssql&link=icon&version=1.0.2' \ -H 'Accept: image/avif,image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8' \ -H 'Accept-Language: en-US,en;q=0.9' \ -H 'Connection: keep-alive' \ -H 'Cookie: R_PCS=light; R_LOCALE=en-us; R_REDIRECTED=true; R_USERNAME=admin; R_THEME=dark; CSRF=0f82d1f94dec5cdeba4397078a69906a; R_SESS=token-7xmb8:kbzwqjjf8w9hp9z7d8888drfj25t7bvjcndtlgg875l67cwbdftbpv' \ -H 'Referer: https://localhost:8443/dashboard/c/local/apps/charts' \ -H 'Sec-Fetch-Dest: image' \ -H 'Sec-Fetch-Mode: no-cors' \ -H 'Sec-Fetch-Site: same-origin' \ -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/110.0.0.0 Safari/537.36' \ -H 'sec-ch-ua: "Chromium";v="110", "Not A(Brand";v="24"' \ -H 'sec-ch-ua-mobile: ?0' \ -H 'sec-ch-ua-platform: "Linux"' \ --compressed \ --insecure

image

Opened a case with DH2i.

The team decided to put the icon on GitHub - https://raw.githubusercontent.com/dh2i/helm/main/assets/DH2i_Logo_Icon.png

Thanks.

Perfect ! Thank you @dh2i-oj
@nicholasSUSE My testing showed the icon for Dxemssql is now displaying correctly in Rancher Apps UI:
dx

Hi @dh2i-oj FYI - As part of this fix, I'm removing Dxemssql's icon url from our upstream.yaml since it's not needed there. See #732