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

Fix bug 1748145: Temporarily remove ability to change contributor email address #2399

Merged
merged 1 commit into from
Jan 6, 2022

Conversation

mathjazz
Copy link
Collaborator

@mathjazz mathjazz commented Jan 6, 2022

Details in the bug.

@mathjazz mathjazz requested a review from eemeli January 6, 2022 20:06
@mathjazz mathjazz merged commit d3f4b0d into mozilla:master Jan 6, 2022
@mathjazz mathjazz deleted the bug-1748145-disable-email-change branch January 6, 2022 20:29
@avm99963
Copy link

avm99963 commented Jan 8, 2022

The bug is private, is this related to a security vulnerability?

I'm asking because as a happy downstream user of Pontoon I'd like to know whether I should apply this patch or the issue is related to the Mozilla deployment. I told a while back my l10n contributors to change their email addresses if they don't want to share it publicly since it is included in Git commits.

Thanks in advance, and thanks for developing such good software! :) I don't have a lot of experience with Python, but if there's anything you might need a hand with, I'd be happy to contribute to Pontoon and also learn some Python on the way!

@mathjazz
Copy link
Collaborator Author

@avm99963 Thanks for reaching out and for the kind words!

Indeed, this patch addresses a security issue. I've filed an issue to establish a proper release process for Pontoon, which should make it easier to follow updates and keep track of the changelog:
#2402

I've also published a post on Discourse about the fix:
https://discourse.mozilla.org/t/security-fix-changing-email-address-temporarily-disabled/91386

@avm99963
Copy link

Thanks @mathjazz! I've just subscribed to the Pontoon category.

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

Successfully merging this pull request may close these issues.

None yet

3 participants