bet-blocker / bet-blocker-api

BetBlockerApi é uma API que fornece uma lista detalhada de hosts de apostas
https://bet-blocker.com
7 stars 2 forks source link

[BUG] Não está retornando corretamente o dns reverso do dominio #5

Closed jhowbhz closed 3 days ago

jhowbhz commented 4 days ago

Repare que no callback está errado o retorno do dns reverso


        "Type": "InterNetwork",
        "Name": "a5sbet.com",
        "Host": "a5sbet.com",
        "ReverseDns": "104.21.42.168",
        "CanonicalName": "a5sbet.com",
        "TTl": "3600",
        "ResolvedAt": "2024-11-09T17:48:37.149205Z"
      },
      "Protocols": {
        "Https": true,
        "Http": true
      },
      "Ips": {
        "Ip": "104.21.42.168",
        "ResolvedAt": "2024-11-09T17:48:37.149204Z"
      },```
brunohash commented 4 days ago

Precisamos fazer um pente fino nesse objeto!

PabloVenino commented 3 days ago

Subi um PR agora, para sanar esse bug. Só precisa daquela revisada :)

brunohash commented 3 days ago

@PabloVenino, obrigado pela contribuição! Ainda sim, precisamos revisar essa DTO pois tem alguns erros na montagem do objeto.

PabloVenino commented 3 days ago

@PabloVenino, obrigado pela contribuição! Ainda sim, precisamos revisar essa DTO pois tem alguns erros na montagem do objeto.

Ai seria melhor quando finalizar o Banco, que vi que estão implementando, o que acha?

RobsonTrasel commented 3 days ago

@PabloVenino, obrigado pela contribuição! Ainda sim, precisamos revisar essa DTO pois tem alguns erros na montagem do objeto.

Ai seria melhor quando finalizar o Banco, que vi que estão implementando, o que acha?

Essa sua mudança já está em vigor pra melhor uso, depois temos que dar uma refactor dentro desses DTO

PabloVenino commented 3 days ago

Sim!! Isso mesmo que quis dizer! Vou acompanhando as issues e vendo com o que posso contribuir :)

RobsonTrasel commented 3 days ago

Show, fechando essa issue porque ja foi resolvida.