Skip to content
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

IDE resources caching from UD is broken #1909

Closed
garagatyi opened this issue Jul 22, 2016 · 2 comments · Fixed by #1931
Closed

IDE resources caching from UD is broken #1909

garagatyi opened this issue Jul 22, 2016 · 2 comments · Fixed by #1931
Assignees
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Milestone

Comments

@garagatyi
Copy link

Earlier UD cached IDE resources to speedup loading of IDE. Now this technick is broken. URL of compilation mapping file is not reachable due to changes in URL scheme. Here is the URL used by UD: http://localhost:8080/ide/_app/compilation-mappings.txt?uid=553185

Reproduction Steps:

  1. Load UD
  2. Check error in console of browser dev tools

Expected behavior:

UD should be able to get compilation mapping file, get resources URL and fetch it into the cache.

Observed behavior:

Request to retrieve compilation mappings file returns main IDE page.
There is an error in console of the browser: Unable to find the IDE javascript file to cache

Che version: 4.6.0-SNAPSHOT
Additional information:

  • Problem can be reliably reproduced, doesn't happen randomly: [Yes]
@garagatyi garagatyi added the kind/bug Outline of a bug - must adhere to the bug report template. label Jul 22, 2016
@TylerJewell TylerJewell added the severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code. label Jul 22, 2016
@TylerJewell TylerJewell added this to the 4.6.0 milestone Jul 23, 2016
@riuvshin
Copy link
Contributor

Is it UD issue?

@benoitf
Copy link
Contributor

benoitf commented Jul 25, 2016

I will fix it

@benoitf benoitf self-assigned this Jul 25, 2016
benoitf added a commit that referenced this issue Jul 25, 2016
Update IDE resource path after changes for namespace handling

Change-Id: I9f935e60be0f461eb2d99fc885a6f23d2b6618be
Signed-off-by: Florent BENOIT <[email protected]>
benoitf added a commit that referenced this issue Jul 25, 2016
Update IDE resource path after changes for namespace handling

Change-Id: I9f935e60be0f461eb2d99fc885a6f23d2b6618be
Signed-off-by: Florent BENOIT <[email protected]>
@benoitf benoitf added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jul 25, 2016
@bmicklea bmicklea mentioned this issue Jul 26, 2016
64 tasks
tolusha pushed a commit that referenced this issue Aug 1, 2016
Update IDE resource path after changes for namespace handling

Change-Id: I9f935e60be0f461eb2d99fc885a6f23d2b6618be
Signed-off-by: Florent BENOIT <[email protected]>
ShimonBenYair pushed a commit to ShimonBenYair/che that referenced this issue Aug 4, 2016
Update IDE resource path after changes for namespace handling

Change-Id: I9f935e60be0f461eb2d99fc885a6f23d2b6618be
Signed-off-by: Florent BENOIT <[email protected]>
skabashnyuk pushed a commit that referenced this issue Jan 3, 2020
Update IDE resource path after changes for namespace handling

Change-Id: I9f935e60be0f461eb2d99fc885a6f23d2b6618be
Signed-off-by: Florent BENOIT <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants