#31599 closed enhancement (fixed)

Create utility class for downloading from HTTP servers

Reported by: karsten Owned by: metrics-team
Priority: Medium Milestone:
Component: Metrics/CollecTor Version:
Severity: Normal Keywords:
Cc: metrics-team Actual Points:
Parent ID: Points:
Reviewer: irl Sponsor:

Description

We have several modules downloading from HTTP servers, which means we have quite some similar-but-not-exactly-the-same code. We should create a utility class, write some tests for that (which might not be trivial, but I don't know yet), and use it from our various modules.

This is something where a friendly volunteer might help!

Child Tickets

Change History (3)

comment:1 Changed 13 months ago by karsten

Reviewer: irl
Status: newneeds_review

No sign of a volunteer, so I volunteered myself. Please review commit 463a721 in my task-31599 branch. Thanks!

comment:2 Changed 13 months ago by irl

Status: needs_reviewmerge_ready

This looks good to me, nice to have the code consolidated and also tested.

comment:3 Changed 13 months ago by karsten

Resolution: fixed
Status: merge_readyclosed

Great, thanks for checking! Merged to master. Closing.

Note: See TracTickets for help on using tickets.