fedora-copr / log-detective-website

Data collection page for Log Detective AI
11 stars 7 forks source link

Submitting a Koji build without cookies ends on a blank page #125

Closed ppisar closed 1 month ago

ppisar commented 2 months ago

I have cookies globally disabled in my web browser. When I submitted a Koji log with a form at https://log-detective.com/#koji, I ended up on a page without any text. After enabling cookies for https://log-detective.com/ I was able to precede.

I recommend adding a warning on that empty page that cookies are required. Similarly you have a warning when JavaScript is disabled.

nikromen commented 2 months ago

that's odd. I thought we don't use cookies for log detective, right @FrostyX ?

FrostyX commented 2 months ago

I thought so as well. I did everything using localStorage. Maybe React uses cookies implicitly for something?

ppisar commented 2 months ago

Indeed. Firefox debugger does not show any Cookies headers transferred when I enable cookies. Maybe Firefox blocks more things than cookies when a permission to Store cookies is Block. Firefox console shows unhelpfully:

Cookie “” has been rejected by user set permissions. 2 [x86_64](https://log-detective.com/contribute/koji/2441874/x86_64)
Request to access cookies or storage on “https://log-detective.com/contribute/koji/2441874/x86_64” was blocked because of custom cookie permission.

Cookie “” has been rejected by user set permissions. [style.css](https://log-detective.com/css/style.css)
Cookie “” has been rejected by user set permissions. [fedora.svg](https://log-detective.com/img/fedora.svg)
Cookie “” has been rejected by user set permissions. [main.js](https://log-detective.com/static/js/main.js)
Cookie “” has been rejected by user set permissions. [x86_64](https://log-detective.com/frontend/contribute/koji/2441874/x86_64)
Cookie “” has been rejected by user set permissions. [favicon.ico](https://log-detective.com/img/favicon.ico)
DOMException: The operation is insecure.
    Pr https://log-detective.com/static/js/main.js:1060
    Pr https://log-detective.com/static/js/main.js:1060
    Qr https://log-detective.com/static/js/main.js:1068
    c https://log-detective.com/static/js/main.js:903
    Fm https://log-detective.com/static/js/main.js:903
    Gm https://log-detective.com/static/js/main.js:905
    lm https://log-detective.com/static/js/main.js:889
    _run https://log-detective.com/static/js/main.js:897
    Gm https://log-detective.com/static/js/main.js:905
    Ue https://log-detective.com/static/js/main.js:129
    Lg https://log-detective.com/static/js/main.js:129
    pi https://log-detective.com/static/js/main.js:135
    jh https://log-detective.com/static/js/main.js:175
    kh https://log-detective.com/static/js/main.js:176
    Pd https://log-detective.com/static/js/main.js:190
    eb https://log-detective.com/static/js/main.js:189
    Id https://log-detective.com/static/js/main.js:187
    Fb https://log-detective.com/static/js/main.js:185
    enqueueForceUpdate https://log-detective.com/static/js/main.js:234
    forceUpdate https://log-detective.com/static/js/main.js:13
    flush_queues https://log-detective.com/static/js/main.js:887
    run_queues https://log-detective.com/static/js/main.js:887
    a https://log-detective.com/static/js/main.js:886
[main.js:148:316](https://log-detective.com/static/js/main.js)
Uncaught DOMException: The operation is insecure.
    Pr https://log-detective.com/static/js/main.js:1060
    Pr https://log-detective.com/static/js/main.js:1060
    Qr https://log-detective.com/static/js/main.js:1068
    c https://log-detective.com/static/js/main.js:903
    Fm https://log-detective.com/static/js/main.js:903
    Gm https://log-detective.com/static/js/main.js:905
    lm https://log-detective.com/static/js/main.js:889
    _run https://log-detective.com/static/js/main.js:897
    Gm https://log-detective.com/static/js/main.js:905
    Ue https://log-detective.com/static/js/main.js:129
    Lg https://log-detective.com/static/js/main.js:129
    pi https://log-detective.com/static/js/main.js:135
    jh https://log-detective.com/static/js/main.js:175
    kh https://log-detective.com/static/js/main.js:176
    Pd https://log-detective.com/static/js/main.js:190
    eb https://log-detective.com/static/js/main.js:189
    Id https://log-detective.com/static/js/main.js:187
    Fb https://log-detective.com/static/js/main.js:185
    enqueueForceUpdate https://log-detective.com/static/js/main.js:234
    forceUpdate https://log-detective.com/static/js/main.js:13
    flush_queues https://log-detective.com/static/js/main.js:887
    run_queues https://log-detective.com/static/js/main.js:887
    a https://log-detective.com/static/js/main.js:886
[main.js:1060](https://log-detective.com/static/js/main.js)