-
Notifications
You must be signed in to change notification settings - Fork 149
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
Add Che Theia editor endpoint to expose the plugins' webviews on a different origin to make them isolated from Che Theia origin #335
Conversation
…fferent origin to make them isolated from Che Theia origin Signed-off-by: Artem Zatsarynnyi <azatsary@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type should be webview instead of ide
targetPort: 3100 | ||
attributes: | ||
protocol: http | ||
type: ide |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Type should be webview instead of ide
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment: eclipse-che/che#15432 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. Thanks @pradeepmalji!
PR has been updated.
Signed-off-by: Artem Zatsarynnyi <azatsary@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Artem Zatsarynnyi azatsary@redhat.com
What does this PR do?
Adds
webviews
endpoint for Che Theia editor plugin to expose the plugins' webviews on a different origin in order to isolate them from Che Theia origin.This PR is needed for the PR eclipse-che/che-theia#591
Note, this PR depends on plugin unique endpoint feature introduced by the PR eclipse-che/che#15432