Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

Update intl_et.arb #1046

Closed
wants to merge 1 commit into from
Closed

Update intl_et.arb #1046

wants to merge 1 commit into from

Conversation

villu164
Copy link

@villu164 villu164 commented Nov 7, 2023

näide.com is not the same as example.com

Read more - https://secret.näide.com/

Summary

Localization was fixed, to use example.com domain not a private domain

Motivation

Security related. Not an urgent(probably seen as a no-issue by many) security issue, but could/should trigger the thought process of the same "example.com" email in other languages

Testing

Compare the output of
https://example.com/
https://näide.com/
https://secret.example.com/
https://secret.näide.com/
To verify that they are not the same

Security fix
Copy link

google-cla bot commented Nov 7, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@villu164
Copy link
Author

Any updates?

@guidezpl
Copy link
Member

Hi, we are turning down the Gallery, and I therefore must close this PR.

@guidezpl guidezpl closed this Jan 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants