-
Notifications
You must be signed in to change notification settings - Fork 566
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
[sync Webclient] Production deployed with build compiled from some hidden place but not from this repository #129
Comments
In open source project, especially security related ones, builds should really be reproducible. Not only matching versions, but file (and libraries) hashes too. |
100% agree, it's a wip :) |
awesome :) |
Oups :/ It's fixed now |
My bad, I forgot to sync. :/ |
Thanks. It's unrelated, but https://beta.protonmail.com/ is not loading with integrity checking errors in dev console. |
@vladimiry there was an issue with the build on beta :/ cf #148 |
voilà ;) |
Friendly reminding (need to update ElectronMail before new release). |
Voilà, sorry we needed to debug something 😁 |
3.16.3 has not been published here. |
ping |
ping (7 days source code publishing lag) |
ping (8 days source code publishing lag) |
@vladimiry done. |
Thanks. Any progress in automating the code publishing process or even better deploying from the public repo? As a temporary supporting solution, I guess an automated notification triggered by new release event could be set up. This notification would inform the concerned parties that the code needs to be published here. |
We are currently refactoring Proton web-apps in React for version 4, all repo are currently public and available there. Our attention is more on the new version than the old Angular repo. For next release on that one, we will add attention to sync repos properly. |
beta 11 is not shared yet but deployed. Besides I guess Please don't miss syncing package lock files. |
v4.0.0-beta13 is live now but not published here yet. Please also publish all the related projects if they have been updated, and lock files too. |
@dhoko do I get it right that the DKIM-related issue detected in beta 16 was somewhere in backend side and that's why I see no changes in client repositories after beta 16 got re-deployed today? Asking since I'm trying to realize if I need to rebuild the WebClient/proton-mail-settings/proton-contacts/proton-calendar event though there was no code update nor in the lock files. |
@vladimiry Yep, we switched back the beta to go back to the 15. The fix is now available on the API side, that's why we deployed the beta 16 again (it's the same bundle we built the first time). No need to rebuild the beta 16, if you did it a few days ago. 👍 |
Deployed production version: 3.15.7
Repository version: 3.15.5
So the project is open-source, but production deployed from some hidden repository.
Expected behavior
To Reproduce
The text was updated successfully, but these errors were encountered: