search

uber / kraken

P2P Docker registry capable of distributing TBs of data in seconds
Apache License 2.0
6.14k stars 423 forks source link

Switch agent to use V2 announce endpoint #292

Closed codygibb closed 4 years ago

codygibb commented 4 years ago

This is long overdue. We added the V2 endpoint to Tracker over a year ago and never migrated the agents over.

By using the V2 endpoint, the info hash will be part of the request URL, and thus our nginx access logs in Tracker will show which info hash is being announced.

codecov[bot] commented 4 years ago

Codecov Report

Merging #292 into master will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #292   +/-   ##
=======================================
  Coverage   64.80%   64.80%           
=======================================
  Files         186      186           
  Lines        7822     7822           
=======================================
  Hits         5069     5069           
  Misses       1999     1999           
  Partials      754      754
Impacted Files Coverage Δ
lib/torrent/scheduler/announcer/announcer.go 75.75% <100.00%> (ø)

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 e435b83...82f5ab6. Read the comment docs.