Skip to content

fix: class cast exception when handling Failed verification result & signature download on Windows #158

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

Merged
merged 4 commits into from
Jul 22, 2025

Conversation

fioan89
Copy link
Collaborator

@fioan89 fioan89 commented Jul 22, 2025

Verification result was improperly cast to download result when signature verification failed to run.
I discovered this issue while porting the signature verifications to Coder Gateway plugin.

Additionally the signature for windows CLI follows the format: coder-windows-amd64.exe.asc
Currently it is coded to coder-windows-amd64.asc which means the plugin always fail to find any signature for windows CLI

fioan89 added 2 commits July 22, 2025 23:15
verification result was improperly cast to download result when signature
verification failed to run.
@fioan89 fioan89 requested review from matifali and f0ssel July 22, 2025 20:17
fioan89 added 2 commits July 22, 2025 23:25
The signature for windows CLI follows the format: coder-windows-amd64.exe.asc
Currently it is coded to coder-windows-amd64.asc which means the plugin
always fail to find any signature for windows cli
@fioan89 fioan89 changed the title fix: class cast exception when handling Failed verification result fix: class cast exception when handling Failed verification result & signature download on Windows Jul 22, 2025
@fioan89 fioan89 requested a review from code-asher July 22, 2025 21:18
@fioan89 fioan89 merged commit 1f23762 into main Jul 22, 2025
6 checks passed
@fioan89 fioan89 deleted the fix-class-cast-exception-when-handling-failed-result branch July 22, 2025 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants