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

Default theme "Innovation" loads font data from Google Fonts servers #1354

Open
domaniko opened this issue Oct 31, 2022 · 7 comments · May be fixed by #1355
Open

Default theme "Innovation" loads font data from Google Fonts servers #1354

domaniko opened this issue Oct 31, 2022 · 7 comments · May be fixed by #1355

Comments

@domaniko
Copy link

domaniko commented Oct 31, 2022

Default theme "Innovation" fetches one font file directly from Google Fonts servers.

There has been a court verdict in Germany that this breaks European data protection law. Some background here. Website owners now get letters from lawyers who claim compensation for violating data protection of their clients (here).

The licence of Google Fonts allows to store Google Font files on local server and add font files also to one's product: https://developers.google.com/fonts/faq.

Best mitigation would be to load the Google font directly from GetSimple installation.

The required change is tiny, but requires one font file to be added to SimpleCMS. On interest I could provide a PR.

@domaniko domaniko changed the title Default theme "Innovation" loads font data from Google Font servers Default theme "Innovation" loads font data from Google Fonts servers Oct 31, 2022
@webketje
Copy link
Contributor

webketje commented Nov 1, 2022

@domaniko thank you so much for bringing this to my attention, I was totally unaware that linking dynamically to a Google Font would breach EU GDPR.

@domaniko
Copy link
Author

domaniko commented Nov 1, 2022

@tablatronix On interest please give me temporarily contributor rights.

The patch can be seen in domaniko@ee29a44

@domaniko
Copy link
Author

domaniko commented Nov 1, 2022

@webketje

@domaniko thank you so much for bringing this to my attention, I was totally unaware that linking dynamically to a Google Font would breach EU GDPR.

Quite many people are very surprised about this harsh interpretation of data protection law by one rather local court. But general advise for the time being is to avoid direct reference to Google Font servers until this is settled,

@tablatronix
Copy link
Member

You can pr anything there is no contributor status

@domaniko domaniko linked a pull request Nov 1, 2022 that will close this issue
@domaniko
Copy link
Author

domaniko commented Nov 1, 2022

PR raised by #1355

@Torsten-K
Copy link

Actually, there is no need to use Google fonts at all. There are free fonts available on fontsquirrel.com for example, which are under the Apache license, like https://www.fontsquirrel.com/fonts/open-sans, and which are allowed to distribute under certain circumstances.

@dimayakovlev
Copy link

No need to bloat standard theme with any additional fonts at all. It can look nice with using only native font stack.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants