Skip to content

Fix: follow redirects when fetching ipinfo db#66

Merged
rennf93 merged 1 commit intorennf93:masterfrom
davidsmfreire:fix/ipinfo-redirect
Nov 16, 2025
Merged

Fix: follow redirects when fetching ipinfo db#66
rennf93 merged 1 commit intorennf93:masterfrom
davidsmfreire:fix/ipinfo-redirect

Conversation

@davidsmfreire
Copy link
Contributor

Description

Let's IPInfoManager follow redirects when dowloading the db.


Related Issue

Fixes #65


Motivation and Context

I think IPInfo changed its API, and it can respond with a 302 code meaning the actual db location is on a redirect URL.


Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation change
  • Performance improvement
  • Code cleanup or refactoring

How Has This Been Tested?

Ran the patch in a production environment.


Screenshots (if appropriate):


Checklist:

  • My code follows the code style of this project (Mypy, Ruff)
  • I have added tests to cover my changes
  • All new and existing tests passed
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have checked that my changes don't introduce any new warnings or errors
  • I have updated the version number if necessary
  • I have added any new dependencies to the appropriate requirements file

@rennf93 rennf93 self-assigned this Nov 4, 2025
@rennf93 rennf93 self-requested a review November 4, 2025 14:46
Copy link
Owner

@rennf93 rennf93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job 💯

@rennf93 rennf93 merged commit 3e9694d into rennf93:master Nov 16, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] IPInfo not following redirect URL

2 participants