Opened 8 weeks ago

Last modified 7 days ago

#28590 merge_ready defect

Replace sbws tests/integration/net.tar with an unpacked version

Reported by: teor Owned by:
Priority: Medium Milestone: sbws: 1.0.x-final
Component: Core Tor/sbws Version:
Severity: Normal Keywords: sbws-1.0-nice-moved-20181128
Cc: juga Actual Points:
Parent ID: Points:
Reviewer: ahf Sponsor:

Description

We don't put binary blobs in our git repositories. It makes diffs impossible. And it's a source of security issues.

Found while tracking down the commit that introduced #28588.

Child Tickets

Change History (9)

comment:1 Changed 8 weeks ago by teor

Milestone: sbws 1.0 (MVP must)sbws 1.0 (MVP nice)

This is nice to have

comment:2 Changed 7 weeks ago by teor

Keywords: sbws-1.0-nice-moved-20181128 added
Milestone: sbws 1.0 (MVP nice)sbws 1.0.4

Moving all sbws 1.0 nice tickets to 1.0.4

comment:3 Changed 7 weeks ago by teor

Milestone: sbws 1.0.4sbws 1.1

Milestone renamed

comment:4 Changed 7 weeks ago by teor

Milestone: sbws 1.1sbws: 1.1.x

Milestone renamed

comment:5 Changed 7 weeks ago by teor

Milestone: sbws: 1.1.xsbws: 1.1.x-final

Milestone renamed

comment:6 Changed 13 days ago by juga

Milestone: sbws: 1.1.x-finalsbws: 1.0.x-final

Moving to 1.0, since it's easy/fast to solve and will avoid continuing to distribute binaries in next release.

comment:7 Changed 13 days ago by juga

Status: newneeds_review

comment:8 Changed 10 days ago by dgoulet

Reviewer: ahf

comment:9 Changed 7 days ago by ahf

Status: needs_reviewmerge_ready

This looks fine as far as I can tell.

  • It looks like the files that have been extracted are the same as in the tarball.
  • I did NOT go over the individual files to see if some of the code or config files could get some clean-up.

Right now the Travis CI fails with what looks like an unrelated GPG failure. Please bump the Travis CI to make it pass before landing, but other than that it looks good to me.

Note: See TracTickets for help on using tickets.