mimblewimble/grin

Replace hyper with a lighterweight framework

Open

#2040 opened on Nov 28, 2018

View on GitHub
 (13 comments) (0 reactions) (0 assignees)Rust (4,876 stars) (991 forks)batch import
enhancementgood first issuehelp wantedquestion

Description

Hyper accounts for close to half of our external dependencies. We need to start weeding those out as much as possible or at least get a good control over them (see #2026).

Suggestions as base, both with far less dependencies:

https://github.com/tomaka/rouille https://github.com/steveklabnik/simple-server (would have to add TLS support)

Note that we could potentially create a fork of those and replace some of their dependencies with ours (same log framework, serde, etc). I think @hashmap and @garyyu have looked a bit at hyper replacements, any feedback?

Contributor guide

Tech stack
rust
Domain
backend
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
4
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
over 1 week
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Rust programming knowledgeFamiliarity with Grin projectUnderstanding of HTTP server frameworksKnowledge of dependency management in Rust
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
20
Research direction
Investigate the two suggested frameworks (rouille, simple server) and compare their dependency trees. Check their TLS support, as simple server would require adding it. Review referenced issue #2026 for context on reducing dependencies. Analyze how the current hyper usage is integrated into Grin's codebase (likely in p2p or API server) and evaluate the effort to port to a new framework, considering shared dependencies like serde and log.