-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ipdatabase.com has shutdown #9
Comments
I will take a look. Thank you
From: Chris Campbell ***@***.***>
Sent: Thursday, June 29, 2023 12:36 PM
To: tevslin/cloudflarepycli ***@***.***>
Cc: Tom Evslin ***@***.***>; Mention ***@***.***>
Subject: Re: [tevslin/cloudflarepycli] ipdatabase.com has shutdown (Issue #9)
@rodmatos <https://github.com/rodmatos> I used this in one of my own projects and noticed this a few days ago. I have created a pull request that removes the dependency on ipdatabase.com that should resolve the problem.
@tevslin <https://github.com/tevslin> could you please review this PR <#10> and let us know you can approve?
—
Reply to this email directly, view it on GitHub <#9 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGLSBBXPI3AHJHX4XMMLRD3XNWVFLANCNFSM6AAAAAAZYV7VOE> .
You are receiving this because you were mentioned. <https://github.com/notifications/beacon/AGLSBBXRXOVR7SYVGCBUTHLXNWVFLA5CNFSM6AAAAAAZYV7VOGWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTAFQRCC.gif> Message ID: ***@***.*** ***@***.***> >
|
closed with version 1.8. removed dependency and added the ability to get the data from cloudflare. Thank you |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Apparently, ipdatabase.com has shutdown and the domain is up for sale.
This leads to the library failing due to the response sent by the webserver. https://github.com/tevslin/cloudflarepycli/blob/29ea4530ac8bf227c4090237ea2f91b8b4730d45/src/cloudflarepycli/cloudflareclass.py#L81-L88
I wonder if it would be possible to remove the dependencies from external services, since Cloudflare should provide all the relevant information.
The text was updated successfully, but these errors were encountered: