-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Terminal is empty on Firefox #13736
Comments
I did some little investigation of this issue. And I see that Firefox blocs not only terminal. It also blocks some svg images(close icons for example, or icons with file type) and maybe some scripts:
TypeError: n.rules is undefined[Learn More] theia.fc4330f36128a73453e5.js:1:1878010
setRules https://assets.openshift.net/che/theia_artifacts/theia.fc4330f36128a73453e5.js:1
onLoad https://assets.openshift.net/che/theia_artifacts/theia.fc4330f36128a73453e5.js:1
o https://assets.openshift.net/che/theia_artifacts/theia.fc4330f36128a73453e5.js:1
setHTML https://assets.openshift.net/che/theia_artifacts/theia.fc4330f36128a73453e5.js:1 This issue is reproducible on the Theia with CDN artifact. Theia which was build without cdn modification works fine. |
@AndrienkoAleksandr I'll look with our CDN / Akamai account contact if he can add the required headers in the files responses returned by the CDN. And will come back to you as soon as I have an answer. |
OTOH when, in Firefox, I disable CSP (with Additionally, even with CSP enabled (=> having the CSP errors on image resources), I'm able to open a working terminal in the That makes me think that there might be several problems at the same time here. @AndrienkoAleksandr Could you also test again in Firefox with |
@davidfestal I will take a look with security.csp.enable=false and will write result of testing. |
@davidfestal I tested terminal with security.csp.enable=false. Terminal works nice. About this one:
I don't think so. Terminal creation is broken, but if you refresh page - it begin work... So security.csp.enable=true really breaks terminal. |
@sunix @AndrienkoAleksandr Could you add in this issue the version of Firefox that produces the errors ? |
My firefox version is 60.6.2esr |
@amisevsk could you confirm what is the precise version of your more recent Firefox, with which you don't seem to have any such problem ? |
I also got this error in Chrome. |
still have the issue @davidfestal |
moving to 7.2.0 according to the firefox related epic - #14015 |
yes it has been @sunix with firefox latest. |
It is working with Che 7.3.0. For Che 7.2.0 and 7.1.0 issue is reproducible for fiirefox 56, 60.4esr, 68.2esr |
Che 7.3.0 I tested with 56, 60.4esr, 68.2esr(latest esr) and latest firefox 70.0, forks fine, so I close issue. Feel free to reopen if it is reproducible with latest Che. |
The text was updated successfully, but these errors were encountered: