Skip to content

Remove rama-net/http dependency from rama-tls crates #745

@soundofspace

Description

@soundofspace

Tls should honestly not know about http in a perfect world. However the real world is far from that so it might be that we close this issue as wont-do.

When removing this dependency we have issues like this:

error[E0432]: unresolved import `rama_net::transport`
  --> rama-tls-boring/src/client/connector.rs:12:15
   |
12 | use rama_net::transport::TryRefIntoTransportContext;
   |               ^^^^^^^^^ could not find `transport` in `rama_net`
   |
  --> rama-tls-boring/src/server/service.rs:18:5
   |
18 |     http::RequestContext,
   |     ^^^^ could not find `http` in `rama_net`
19 |     tls::{ApplicationProtocol...
20 |     transport::TransportContext,
   |     ^^^^^^^^^ could not find `transport` in `rama_net`

On first sight the only issues seems to be around TransportContext and RequestContext and those will be reworked in this issue, so after that it might be possible to just remove this dependency without extra work

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions