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

Bumps tor version 0.3.5.8 -> 0.4.0.5 #4661

Merged
merged 1 commit into from
Aug 9, 2019
Merged

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented Aug 7, 2019

Status

Ready for review.

Description of Changes

Fixes #4658.

Changes proposed in this pull request:

Updates the fetch-tor-packages logic to pull in current stable LTS [0]
versions of Tor. The prior 0.3.x versions are no longer available from
upstream tor repo, so they cannot be fetched.

Includes updates to test vars, as well.

[0] https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/CoreTorReleases

Testing

We should also confirm that tor-over-ssh works fine on 0.4.x, but that can wait until dedicated QA period.

Deployment

Yes, significant version change to tor. Must be tested thoroughly prior to release of SD 1.0.0.

Checklist

If you made changes to the server application code:

  • Linting (make lint) and tests (make -C securedrop test) pass in the development container

If you made changes to securedrop-admin:

  • Linting and tests (make -C admin test) pass in the admin development container

If you made changes to the system configuration:

If you made non-trivial code changes:

  • I have written a test plan and validated it for this PR

If you made changes to documentation:

  • Doc linting (make docs-lint) passed locally

Updates the fetch-tor-packages logic to pull in current stable LTS [0]
versions of Tor. The prior 0.3.x versions are no longer available from
upstream tor repo, so they cannot be fetched.

Includes updates to test vars, as well.

[0] https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/CoreTorReleases
@kushaldas
Copy link
Contributor

@conorsch I just now approved the PR for the git LFS repository, and then sadly figured out that I don't have merge rights in that repository.

@conorsch
Copy link
Contributor Author

conorsch commented Aug 8, 2019

sadly figured out that I don't have merge rights in that repository

Fixed! Please try again, @kushaldas.

@kushaldas
Copy link
Contributor

@conorsch worked :) Now I will wait for 15 minutes and then restart the CI job.

@codecov-io
Copy link

codecov-io commented Aug 8, 2019

Codecov Report

Merging #4661 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #4661   +/-   ##
========================================
  Coverage    82.67%   82.67%           
========================================
  Files           45       45           
  Lines         3122     3122           
  Branches       338      338           
========================================
  Hits          2581     2581           
  Misses         454      454           
  Partials        87       87

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 20cbd52...0cbd30b. Read the comment docs.

@kushaldas
Copy link
Contributor

Kicking the Python2 app tests.

Copy link
Contributor

@kushaldas kushaldas left a comment

Choose a reason for hiding this comment

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

Finally the CI is green. Approved 🌈

@kushaldas kushaldas merged commit de8cbcf into develop Aug 9, 2019
@conorsch conorsch deleted the 4658-update-tor-to-0.4.x branch June 18, 2020 21:54
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.

Move to tor 0.4.x release series
3 participants