CloudinaryException is too vague

Aberta
#338 3 comentários 1 reação 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
35/100
Tipo de issue
Funcionalidade
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
ruby
Domínio
api, backend

Direção de pesquisa

Localize CloudinaryException e o tratamento das respostas HTTP que atualmente produz mensagens como "Resource not found" e "Error in loading.". Rastreie como as respostas 400–499 são classificadas, depois defina a hierarquia de exceções e verifique se os erros de cliente relevantes podem ser capturados separadamente. A issue não menciona arquivos ou testes específicos, portanto a cobertura existente de testes de exceções e do cliente HTTP deve ser identificada primeiro.

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

Descrição

enhancement

We've seen messages like Resource not found and Error in loading, which we'd like to handle as generic HTTP client errors (400-499). Unfortunately, we have to rescue the CloudinaryException, check its error message, and then determine what should be done.

I would propose having more than one exception class to make it possible for us to rescue something like Cloudinary::HttpClientError to ignore errors like HTTP 403 (Forbidden), 404 (Not Found), 408 (Request Timeout), etc.

One way to do this would be to add class Cloudinary::HttpClientError < CloudinaryException; end and then use that error for the cases that represent HTTP 400-499.

Linguagem predominante
Ruby
Estrelas
420
Forks
282
Métricas de merge de PRs
Nenhum PR com merge em 30d

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 cloudinary/cloudinary_gem

Todas as issues de cloudinary/cloudinary_gem

Issues semelhantes

Mais issues de Ruby

Receba novas issues na sua caixa de entrada

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