Support Certifi 2.19.x

Aberta Para iniciantes
  • Algo errado?
#1,028 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
70/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
elixir, erlang
Domínio
networking

Direção de pesquisa

Comece pela restrição de dependência do certifi que o Hackney 4.11.0 declara, atualmente ~> 2.18.0, e verifique se ela é intencional. Amplie-a somente depois de confirmar que os caminhos de código relacionados a certificados continuam funcionando com o Certifi 2.19.0, e adicione cobertura de regressão, se necessário. Está concluído quando a reprodução com Mix resolve tanto hackney 4.11.0 quanto certifi 2.19.0, e rebar3 eunit passa.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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!

Linguagem predominante
Erlang
Estrelas
1.4k
Forks
458
Merge médio
11h 32min
PRs com merge (30d)
47

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de benoitc/hackney

Todas as issues de benoitc/hackney

Issues semelhantes

Mais issues de Networking

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.