version 7287 - feat GIPHY integration leads to loading woff2 font files from third party (aws) - CSP is required to block the requests - again
See original GitHub issueDescription: Since version 7210 / 7287 the resuling build attempts to load woff2 fonts from amazonaws. It is blocked by CSP settings, but this shouldnt happen.
My previous Issue: https://github.com/jitsi/jitsi-meet/issues/11419 -> was closed and fixed -> waited for next release but it seems the issue still happens (again?)
Fixed in https://github.com/jitsi/jitsi-meet/pull/11457 -> but seems to happen again
Its probably related to these errors i get when building?
> [jitsi-meet@0.0.0](mailto:jitsi-meet@0.0.0) postinstall
> patch-package && jetify
patch-package 6.4.7
Applying patches...
[eslint-plugin-flowtype@8.0.3](mailto:eslint-plugin-flowtype@8.0.3) ✔
[react-native@0.68.1](mailto:react-native@0.68.1) ✔
[react-native-dialog@9.2.1](mailto:react-native-dialog@9.2.1) ✔
[react-native-gesture-handler@2.1.0](mailto:react-native-gesture-handler@2.1.0) ✔
**ERROR** Failed to apply patch for package @giphy/js-analytics at path
node_modules/@giphy/js-analytics
This error was caused because @giphy/js-analytics has changed since you
made the patch file for it. This introduced conflicts with your patch,
just like a merge conflict in Git when separate incompatible changes are
made to the same piece of code.
Maybe this means your patch file is no longer necessary, in which case
hooray! Just delete it!
Otherwise, you need to generate a new patch file.
To generate a new one, just repeat the steps you made to generate the first
one.
i.e. manually make the appropriate file changes, then run
patch-package @giphy/js-analytics
Info:
Patch file: [patches/@giphy+js-analytics+4.0.7.patch](mailto:patches/@giphy+js-analytics+4.0.7.patch)
Patch was made for version: 4.0.7
Installed version: 4.1.0
**ERROR** Failed to apply patch for package @giphy/js-brand at path
node_modules/@giphy/js-brand
This error was caused because @giphy/js-brand has changed since you
made the patch file for it. This introduced conflicts with your patch,
just like a merge conflict in Git when separate incompatible changes are
made to the same piece of code.
Maybe this means your patch file is no longer necessary, in which case
hooray! Just delete it!
Otherwise, you need to generate a new patch file.
To generate a new one, just repeat the steps you made to generate the first
one.
i.e. manually make the appropriate file changes, then run
patch-package @giphy/js-brand
Info:
Patch file: [patches/@giphy+js-brand+2.0.4.patch](mailto:patches/@giphy+js-brand+2.0.4.patch)
Patch was made for version: 2.0.4
Installed version: 2.1.0
---
patch-package finished with 2 error(s).
https://s3.amazonaws.com/giphyscripts/react-giphy-brand/fonts/InterFace_W_Rg.woff2 https://s3.amazonaws.com/giphyscripts/react-giphy-brand/fonts/InterFace_W_Rg.woff …
Steps to reproduce: Open the meet site with the build in question, Loading welcome page suffices Expected behavior: No attempts to load fonts or files from third party sites.
Actual behavior: attempts to load .woff/.woff2 fonts from aws
Server information: Jitsi Meet version: 7287 Operating System: Linux Client information: Operating System: Linux
Issue Analytics
- State:
- Created a year ago
- Comments:7 (4 by maintainers)
Top GitHub Comments
That is very good to know! Thanks, removing that step right now 👍
The error patching the packages? That means you have updated giphy on your own clone. Don’t.