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

Investigate signature verification #67

Closed
code-asher opened this issue Oct 28, 2024 · 3 comments
Closed

Investigate signature verification #67

code-asher opened this issue Oct 28, 2024 · 3 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists question Further information is requested

Comments

@code-asher
Copy link
Member

code-asher commented Oct 28, 2024

Got a report that the marketplace does not work for 1.94 because of signature verification.

EDIT: See #65 instead

@coder-labeler coder-labeler bot added bug Something isn't working question Further information is requested labels Oct 28, 2024
@code-asher
Copy link
Member Author

code-asher commented Oct 28, 2024

I think we will need to implement https://github.com/filiptronicek/node-ovsx-sign in Go. We generate what we need when an extension is added, or on demand for existing extensions for backwards compatibility.

@code-asher code-asher changed the title Investigation signature verification Investigate signature verification Oct 28, 2024
@janLo
Copy link
Contributor

janLo commented Oct 29, 2024

Does VS Code then need to get the public key of the signature configured somewhere?

As mentioned in #65 we currently just download and provide the signature from the upstream marketplace and get a config-free experience with that (we have an air-gapped deployment so we could also just point the marketplace.visualstudio.com URL to our reverse proxy so that we don't even need to change that in the product.json).

I would rather keep this ability of being able to fake the upstream as much as possible to reduce friction on the user's end.

@code-asher
Copy link
Member Author

Ah, thanks for pointing out that issue, I was away last week and still have to go through my backlog. I will close this as a duplicate.

I am not sure about the public key question; this is part of what needs to be investigated. I know Open VSX has solved this though, so I do not think it requires any changes on the VS Code side.

But, if it is important to keep the ability to add your own signatures, we can make that work.

@code-asher code-asher closed this as not planned Won't fix, can't repro, duplicate, stale Oct 29, 2024
@code-asher code-asher added the duplicate This issue or pull request already exists label Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants