[GH-ISSUE #653] [Tokio migration] Proxy support #381

Closed
opened 2026-02-27 19:30:18 +03:00 by kerem · 1 comment
Owner

Originally created by @Johannesd3 on GitHub (Feb 25, 2021).
Original GitHub issue: https://github.com/librespot-org/librespot/issues/653

Proxy support was one of the few things that weren't easy to migrate to tokio 1.0, mainly because hyper-proxy is not migrated yet. While #583 added basic proxy support again, it was only tested with a quickly set up local proxy server instance. In the discussion of this PR, 3 alternatives came up (read there for their advantages an drawbacks):

  1. Using the simple proxytunnel
  2. Waiting for hyper-proxy
  3. Using reqwest

Since I don't know anything about proxy servers (and don't need them), I would appreciate if someone else takes a look at it.

@sashahilton00 Would it be possible to use milestones or custom labels to keep track of the issues related to the tokio migration?

Originally created by @Johannesd3 on GitHub (Feb 25, 2021). Original GitHub issue: https://github.com/librespot-org/librespot/issues/653 Proxy support was one of the few things that weren't easy to migrate to tokio 1.0, mainly because hyper-proxy is not migrated yet. While #583 added basic proxy support again, it was only tested with a quickly set up local proxy server instance. In the discussion of this PR, 3 alternatives came up (read there for their advantages an drawbacks): 1. Using the simple proxytunnel 2. Waiting for hyper-proxy 3. Using reqwest Since I don't know anything about proxy servers (and don't need them), I would appreciate if someone else takes a look at it. @sashahilton00 Would it be possible to use milestones or custom labels to keep track of the issues related to the tokio migration?
kerem 2026-02-27 19:30:18 +03:00
  • closed this issue
  • added the
    Tokio 1.0
    label
Author
Owner

@Johannesd3 commented on GitHub (Mar 8, 2021):

They did update hyper-proxy, so we can make progress here.

BTW, while re-implementing proxy support I did not really care for the format of addresses (e.g. whether port or protocol are specified), I think the behaviour could differ compared to before. We should check this.

<!-- gh-comment-id:793115137 --> @Johannesd3 commented on GitHub (Mar 8, 2021): They did update hyper-proxy, so we can make progress here. BTW, while re-implementing proxy support I did not really care for the format of addresses (e.g. whether port or protocol are specified), I think the behaviour could differ compared to before. We should check this.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/librespot#381
No description provided.