Support Certifi 2.19.x

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#1,028 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
70/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
elixir, erlang
Lĩnh vực
networking

Hướng nghiên cứu

Bắt đầu từ yêu cầu phụ thuộc certifi mà Hackney 4.11.0 khai báo, hiện là ~> 2.18.0, và kiểm tra xem nó có chủ đích hay không. Chỉ nới rộng sau khi xác nhận các đường dẫn mã liên quan đến chứng chỉ vẫn hoạt động với Certifi 2.19.0, và thêm kiểm thử hồi quy nếu cần. Hoàn thành khi việc tái hiện bằng Mix phân giải được cả hackney 4.11.0 và certifi 2.19.0, và rebar3 eunit vượt qua.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Hi! Thank you for maintaining Hackney. Would you consider updating or widening its Certifi requirement to support Certifi 2.19.0, after checking compatibility?

Hackney 4.11.0 currently requires certifi ~> 2.18.0, which excludes 2.19.0. I encountered this while refreshing a downstream application's dependencies; an application selecting Certifi 2.19.0 cannot currently resolve it alongside Hackney 4.11.0.

Minimal reproduction

In an otherwise empty Mix project, use:

defmodule DependencyProbe.MixProject do
  use Mix.Project

  def project do
    [
      app: :dependency_probe,
      version: "0.1.0",
      deps: [
        {:hackney, "== 4.11.0"},
        {:certifi, "== 2.19.0"}
      ]
    ]
  end
end

Run mix deps.get. I reproduced the resolution failure on Elixir 1.20.4 / OTP 29: Hackney's ~> 2.18.0 requirement excludes the requested Certifi release.

Published requirements: Hackney 4.11.0.

Request and offer to help

Would a PR for Certifi 2.19 compatibility be welcome? I'd be happy to investigate any required changes, update the dependency requirement, add or adjust certificate-related regression coverage, and run rebar3 eunit. If the current restriction is intentional, I'd appreciate any guidance on what would be needed to support the newer release.

Thanks for your time and for considering this!

Ngôn ngữ chính
Erlang
Star
1.4k
Fork
458
Merge trung bình
11 giờ 32 phút
Pull request đã merge (30 ngày)
47

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của benoitc/hackney

Tất cả issue của benoitc/hackney

Issue tương tự

Thêm issue về Networking

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.