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

Protobuf gencode version 5.27.2 is older then the runtime version 5.28.1 at playwright.proto #3789

Closed
Houbein opened this issue Sep 14, 2024 · 7 comments
Labels
bug Something isn't working priority: high
Milestone

Comments

@Houbein
Copy link

Houbein commented Sep 14, 2024

Describe the bug
In version 18.8.1 rfbrowserinit fails with the following error:
Failed to download Chromium 129.0.6668.29 (playwright build v1134) caused by Download failure, code=1
at ChildProcess. (c:\Python312\Lib\site-packages\Browser\wrapper\node_modules\playwright-core\lib\server\registry\browserFetcher.js:91:16)

There is also a warning:
c:\Python312\Lib\site-packages\google\protobuf\runtime_version.py:122: UserWarning: Protobuf gencode version 5.27.2 is older then the runtime version 5.28.1 at playwright.proto. Please avoid checked-in Protobuf gencode that can be obsolete.

To Reproduce
Install version 18.8.1
run rfbrowserinit

Desktop:

  • OS: Windows 10 22h2

Additional context
Downgrading to 18.8.0 solves the issue.

@aaltat
Copy link
Member

aaltat commented Sep 14, 2024

The download is controlled by Playwright and we can’t fix it. Because we run Windows also in our CI, the binary should be there, have you retried the installation and see what it says?

The protobuf warning is Browser library problem and I did notice it too after the release. Need to see how it can be fixed.

@Houbein
Copy link
Author

Houbein commented Sep 14, 2024

Cause found, 18.8.1 calls for chromium 1134 which seems to have not been picked up by our internal mirror yet.
18.8.0 uses 1129 which is present.

@aaltat aaltat changed the title rfbrowserinit fails in 18.8.1 Protobuf gencode version 5.27.2 is older then the runtime version 5.28.1 at playwright.proto. Please avoid checked-in Protobuf gencode that can be obsolete. Sep 14, 2024
@aaltat aaltat added bug Something isn't working priority: high labels Sep 14, 2024
@aaltat aaltat added this to the v18.9.0 milestone Sep 14, 2024
@aaltat
Copy link
Member

aaltat commented Sep 14, 2024

@allcontributors please add @Houbein for bugs

Copy link
Contributor

@aaltat

I've put up a pull request to add @Houbein! 🎉

@aaltat
Copy link
Member

aaltat commented Sep 26, 2024

I think this is caused by protocolbuffers/protobuf#18096 and we need to wait for new release from protobuf.

@Snooz82
Copy link
Member

Snooz82 commented Oct 7, 2024

We fixed it so, that we have now downgraded the protobuf version.

@Snooz82 Snooz82 closed this as completed Oct 7, 2024
@Snooz82 Snooz82 changed the title Protobuf gencode version 5.27.2 is older then the runtime version 5.28.1 at playwright.proto. Please avoid checked-in Protobuf gencode that can be obsolete. Protobuf gencode version 5.27.2 is older then the runtime version 5.28.1 at playwright.proto Oct 7, 2024
@Houbein
Copy link
Author

Houbein commented Oct 10, 2024

I understand, but I don't really consider a downgrade to be a "fix".
I know I'm a bit of an LCM freak (ask my team), but in my opinion the only path ever should be up.
So I do not consider this issue to be fixed until a new version of protobuf is implemented.
Always forward, never backwards!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority: high
Projects
None yet
Development

No branches or pull requests

3 participants