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

Wordlist removals and translation updates for next SecureDrop release #5895

Merged
merged 2 commits into from
Jun 8, 2021

Conversation

eloquence
Copy link
Member

@eloquence eloquence commented Apr 9, 2021

Status

Ready for review. See previously: #5357, #1546; 366c657.

Description of Changes

  • Removes potentially offensive words (alone or in combination) from the SecureDrop wordlists. Among other things, SecureDrop uses adjectives and nouns to generate journalist designations, and while we cannot avoid all potentially offensive combinations, we can exclude some words that are almost always inappropriate in journalist designations or passphrases.

  • Addresses translation feedback by pappasadrian and KwadroNaut from https://weblate.securedrop.org/translate/securedrop/securedrop/en/?checksum=e3e9531df602610b

@eloquence eloquence marked this pull request as ready for review June 3, 2021 04:50
@eloquence eloquence requested a review from a team as a code owner June 3, 2021 04:50
@eloquence eloquence changed the title Wordlist removals for next SecureDrop release Wordlist removals and translation updates for next SecureDrop release Jun 8, 2021
@eloquence eloquence force-pushed the wordlist-removals branch from c6eb5d2 to 8d3658a Compare June 8, 2021 00:24
@eloquence
Copy link
Member Author

Since it's a small change, I've pulled in one low-hanging fruit string improvement as well, suggested in https://weblate.securedrop.org/translate/securedrop/securedrop/en/?checksum=e3e9531df602610b, as part of an overall review of Weblate comments (#5869).

@eloquence eloquence force-pushed the wordlist-removals branch from 8d3658a to c16c952 Compare June 8, 2021 00:39
@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2021

Codecov Report

Merging #5895 (c16c952) into develop (74a84f3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #5895   +/-   ##
========================================
  Coverage    85.12%   85.12%           
========================================
  Files           55       55           
  Lines         3865     3865           
  Branches       480      480           
========================================
  Hits          3290     3290           
  Misses         463      463           
  Partials       112      112           
Impacted Files Coverage Δ
securedrop/journalist_app/utils.py 82.07% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 74a84f3...c16c952. Read the comment docs.

Copy link
Contributor

@zenmonkeykstop zenmonkeykstop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM based on visual review, CI results.

@zenmonkeykstop zenmonkeykstop merged commit 3d46171 into develop Jun 8, 2021
@rmol rmol deleted the wordlist-removals branch June 23, 2021 13:51
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.

3 participants