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

Reset master to release #5203

Merged
merged 3,617 commits into from
Apr 21, 2020
Merged

Reset master to release #5203

merged 3,617 commits into from
Apr 21, 2020

Conversation

zenmonkeykstop
Copy link
Contributor

Status

Ready for review

Description of Changes

Addresses #5081.

Updates master to reflect the latest release branch, to allow it to be used as a stable point for ReadTheDocs-hosted documentation.

Testing

  • the resultant master branch is identical to release/1.2.2?

Conor Schaefer and others added 30 commits September 27, 2019 12:47
During review of #4865, I misinterpreted the new checkboxes as a
sequential list, rather than an either/or choice. Adding a small
clarification to prevent future misunderstandings. =)
Updates pip dependencies: Ansible, Pyyaml, Jinja2, Molecule, Werkzeug
This will inform the admins to rerun ./securedrop-admin setup
as this will create .venv3 dirctory for Python3 dependencies.
As Tor4 has Python 3.7.3, we should just mention without any
particular version.
A smattering of the familiar Python 2 -> 3 changes, mostly around string
handling, filepaths to interpreters and virtualenvs, and syntax updates.
Specifically:

  * coerces major release number to int
  * converts iteritems() -> items() in dynamic inventory
  * updates string handling in dynamic inventory
  * ensures securedrop_init runs under python3

Tested under Tails 3.16 (based on Stretch), running Python 3.5. Worked
well, after these fixes landed, although I've observed a handful of the
"weakref" stderr output. The weakref output doesn't break anything, but
it may be alarming to Admins who haven't updated to Tails 4 & Python 3.7
yet.
Enables Source and Journalist desktop icons in Tails 4
Improve specification of securedrop-app-code dependencies
- The "Get Tor" instructions as written didn't work; the email
  autoresponder requires the OS to be specified, and the GitHub
  downloads are currently not available.

- The instructions suggested visiting the .onion address _before_
  "choosing who to submit to".

- Added a recommendation to verify the .onion address against the
  SecureDrop Directory

- Used more common terms in a couple of places (e.g.,
  "USB Flash Drive" instead of "USB Key"), simplified Flag for
  Reply language

- Wrap to 80 characters
Several instances of the warning below:

[DEPRECATION WARNING]: Invoking "apt" only once while using a loop via
squash_actions is deprecated. Instead of using a loop to supply multiple items
and specifying `name: "{{ item }}"`, please use `name: '{{ unused_packages }}'`
 and remove the loop. This feature will be removed in version 2.11. Deprecation
 warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
…assword

Added check for admin password in GUI updater
contributors:
  1000101
  Michal Stanke
  darmozrac
  michaela-bot

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 570f297
Also update the translation-test script so it gets the list of
supported locales from i18n_tool.py and will no longer have to be
updated when we change that list.
Edits for readability and concision. Also remove last of language
related to RTD migration.
Edit README section about documentation
These requirements were installed via pip in the Dev/Admin Docker containers. We should instead add them to the develop-requirements file and pin hashes.

The `--allow-unsafe` flags should not impact security: it is to allow pip to pin setuptools and pip-tools (see issue #4686).
Also adds --allow-unsafe to allow pinning hashes of setuptools in test-requirements.txt

The `--allow-unsafe` flags should not impact security: it is to allow pip to pin setuptools and pip-tools (see issue #4686).
Certain combinations of language and viewport size could result in the
two columns on the source start page not lining up with each
other. This changes the columns' justify-content property to push the
top and bottom containers apart, and controls the borders and content
of the buttons to align them better.
…n-requirements

Pin all dev and testing dependencies
Add Czech to list of supported languages
The page moved to a new URL without a redirect
Bump app tests timeout to 15 minutes
zenmonkeykstop and others added 24 commits February 14, 2020 16:08
(cherry picked from commit 63f88f7)
(Rounded corners edited out as they have a glitchy looking fringe
outside the context in which the screenshot was taken)

(cherry picked from commit a128e49)
[1.2.1] Add 1.2.0 to 1.2.1 update guide for workstations
(cherry picked from commit 1f20b71)
(cherry picked from commit 5dad266)
Build image needed updates

(cherry picked from commit 33b252c)
[1.2.2] Backport 5157, 5158 and 5159 (and 5151, 5154 for CI to pass)
(cherry picked from commit 5fa2b49)
@lgtm-com
Copy link

lgtm-com bot commented Apr 21, 2020

This pull request introduces 14 alerts and fixes 9 when merging 93f5212 into 2350afb - view on LGTM.com

new alerts:

  • 4 for Implicit string concatenation in a list
  • 3 for Except block handles 'BaseException'
  • 2 for Unused local variable
  • 2 for Unused import
  • 1 for Unmatchable dollar in regular expression
  • 1 for Testing equality to None
  • 1 for Variable defined multiple times

fixed alerts:

  • 5 for Unused import
  • 4 for Except block handles 'BaseException'

Copy link
Contributor

@redshiftzero redshiftzero left a comment

Choose a reason for hiding this comment

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

git diff release/1.2.2 reset-master-to-release looks good, CI will fail here but that's fine as master branch will be used for the purpose of building documentation only

@redshiftzero redshiftzero merged commit 8b8c8e1 into master Apr 21, 2020
@zenmonkeykstop zenmonkeykstop deleted the reset-master-to-release branch May 7, 2020 01:31
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.