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

Add marketplace.json #380

Merged
merged 1 commit into from
Jul 27, 2022
Merged

Conversation

filiptronicek
Copy link
Member

@filiptronicek filiptronicek commented Jul 11, 2022

This PR sets the URL for deprecated, malicious and other special extensions in product.json with values fit for https://open-vsx.org. It would be great to have it under Eclipse, but for now it's under https://github.com/gitpod-io/gitpod/blob/main/components/ide-proxy/static/code/marketplace.json.

This PR fixes gitpod-io/gitpod#10847.

How to test

  1. Open this PR in Gitpod
  2. Wait for the new window of VS Code to open
  3. Search for KnisterPeter.vscode-github in the extensions tab
  4. Make sure there is no install button

@filiptronicek filiptronicek added the enhancement New feature or request label Jul 11, 2022
@filiptronicek filiptronicek self-assigned this Jul 11, 2022
@jeanp413
Copy link
Member

jeanp413 commented Jul 11, 2022

Great 🎉 but I have one question: Do we need to take into account selfhosted too or not? is it Ok to assume they have acces to Github? cc @filiptronicek @akosyakov

@filiptronicek
Copy link
Member Author

Great 🎉 but I have one question: Do we need to take into account selfhosted too or not? is it Ok to assume they have acces to Github?

I think we should take them into account, @jeanp413, good point about the GitHub access. For them maybe we can (and do this for SaaS as well) just wget the file from Github so it ends up in the IDE image and we can request it through blobserve every time that image gets rebuilt, although maybe for critical updates (newly found malicious extensions) that's not fast enough :/. Or somehow proxy the file URL?

Also we should only include this for self-hosted if they don't self-host Open VSX, because otherwise there might be errors.

@akosyakov
Copy link
Member

Why don't we put it IDE proxy as a static resource? Later IDE proxy can just forward connections to OpenVSX.

@filiptronicek
Copy link
Member Author

Why don't we put it IDE proxy as a static resource? Later IDE proxy can just forward connections to OpenVSX.

If we did that, @akosyakov, would the file still point to the GitHub file (i.e. could we change the file after deployment by updating it on GitHub)? I think that is pretty important as I mentioned with self hosted who perhaps don't deploy so often when there are new malicious extensions.

@akosyakov
Copy link
Member

akosyakov commented Jul 13, 2022

If we did that, @akosyakov, would the file still point to the GitHub file (i.e. could we change the file after deployment by updating it on GitHub)? I think that is pretty important as I mentioned with self hosted who perhaps don't deploy so often when there are new malicious extensions.

I don't think posting on GitHub is good solution for Self-Hosted. It should be either within their network, since they can always can connect to it, i.e. on IDE proxy, or fetched from OpenVSX. GitHub is not necessary available to them. So for now putting it in IDE proxy directly is a step in right direction.

@filiptronicek
Copy link
Member Author

@akosyakov or we could add this to Open VSX proxy as an endpoint 🤔 (this means we could afterwards also modify the GitHub URL to point to Eclipse instead of us)

@akosyakov
Copy link
Member

@akosyakov or we could add this to Open VSX proxy as an endpoint 🤔 (this means we could afterwards also modify the GitHub URL to point to Eclipse instead of us)

IDE proxy is interface of IDE team to the world, OpenVSX proxy is behind IDE proxy as well to forward traffic. It does not have other responsibilities.

@filiptronicek
Copy link
Member Author

For now, I agree we can just pull the file as a static asset with IDE Proxy. I will make a PR in https://github.com/gitpod-io/gitpod for that.

After and if we push EclipseFdn/open-vsx.org#1121 forward, Eclipse can maybe proxy the file via https://open-vsx.org so it will be available through Open VSX Proxy automatically.

@filiptronicek
Copy link
Member Author

I have made gitpod-io/gitpod#11361 which adds this to our proxy. You can try this out in the preview environment: https://ft-add-ext649a97d67d.preview.gitpod-dev.com/workspaces and I have confirmed that the rewrite works correctly. Anything else you think we should do before merging and deploying, @akosyakov & @jeanp413?

@akosyakov akosyakov force-pushed the gp-code/main branch 2 times, most recently from bb66fbc to 8225b8c Compare July 15, 2022 23:08
@akosyakov akosyakov requested a review from jeanp413 July 18, 2022 15:31
@jeanp413
Copy link
Member

Looks good need to wait for gitpod-io/gitpod#11361 to be merged and deployed

@akosyakov akosyakov force-pushed the gp-code/main branch 3 times, most recently from 99a52e9 to 8649946 Compare July 20, 2022 23:10
@filiptronicek filiptronicek force-pushed the ft/add-control-url-marketplace branch from 5c76410 to 3fef802 Compare July 22, 2022 09:43
@jeanp413 jeanp413 force-pushed the gp-code/main branch 2 times, most recently from aede784 to aade49c Compare July 27, 2022 05:01
@filiptronicek filiptronicek force-pushed the ft/add-control-url-marketplace branch from 3fef802 to 6dc2cf1 Compare July 27, 2022 12:56
@filiptronicek filiptronicek force-pushed the ft/add-control-url-marketplace branch from 6dc2cf1 to 5109648 Compare July 27, 2022 12:59
@filiptronicek
Copy link
Member Author

@jeanp413 could you please merge? 🙏

@jeanp413 jeanp413 merged commit 0085d92 into gp-code/main Jul 27, 2022
@jeanp413 jeanp413 deleted the ft/add-control-url-marketplace branch July 27, 2022 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants