Skip to content

Added ip2location.io for IP geolocation lookup. #11170

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.

Sign up for GitHub

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

Jump to bottom
Open
ip2location wants to merge 26 commits into
base: main
Choose a base branch
from
Open

Added ip2location.io for IP geolocation lookup. #11170

ip2location wants to merge 26 commits into from
+213 −0

Conversation

@ip2location
Copy link

@ip2location ip2location commented Nov 17, 2025

SUMMARY

Added new IP geolocation lookup using the free IP2Location.io API service.

ISSUE TYPE
  • New Module/Plugin Pull Request
COMPONENT NAME

ip2locationio

@ansibullbot ansibullbot added module module new_contributor Help guide this first time contributor plugins plugin (any type) labels Nov 17, 2025
@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-12 Automatically create a backport for the stable-12 branch labels Nov 17, 2025
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot ansibullbot added ci_verified Push fixes to PR branch to re-run CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Nov 17, 2025
@ansibullbot ansibullbot removed ci_verified Push fixes to PR branch to re-run CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Nov 17, 2025
felixfontein
felixfontein reviewed Nov 21, 2025
View reviewed changes
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Please note that you need to use the code formatter, and that you need to add tests. Please see CONTRIBUTING.md.

ip2location and others added 7 commits November 24, 2025 07:23
@ip2location @felixfontein
Update plugins/modules/ip2locationio_facts.py
54bcad8 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2locationio_facts.py
d34c8eb 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2locationio_facts.py
561606e 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2locationio_facts.py
81c39a3 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2locationio_facts.py
f3a7b6c 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2locationio_facts.py
a6be241 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2locationio_facts.py
b7d3650 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot ansibullbot added ci_verified Push fixes to PR branch to re-run CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Nov 23, 2025
@ansibullbot ansibullbot removed the ci_verified Push fixes to PR branch to re-run CI label Nov 24, 2025
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot ansibullbot added the ci_verified Push fixes to PR branch to re-run CI label Nov 24, 2025
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot
Copy link
Collaborator

ansibullbot commented Nov 24, 2025

@ip2location this PR contains the following merge commits:

Please rebase your branch to remove these commits.

click here for bot help

@ansibullbot ansibullbot added merge_commit This PR contains at least one merge commit. Please resolve! needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html and removed ci_verified Push fixes to PR branch to re-run CI labels Nov 24, 2025
@ansibullbot ansibullbot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR label Nov 24, 2025
@felixfontein
Copy link
Collaborator

felixfontein commented Nov 29, 2025

Please don't forget to add tests (and check out CONTRIBUTING.md). Thanks.

@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot ansibullbot added ci_verified Push fixes to PR branch to re-run CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR new_plugin New plugin labels Nov 30, 2025
@ansibullbot ansibullbot removed ci_verified Push fixes to PR branch to re-run CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Dec 1, 2025
felixfontein
felixfontein reviewed Dec 1, 2025
View reviewed changes
russoz
russoz reviewed Dec 1, 2025
View reviewed changes
Copy link
Collaborator

@russoz russoz left a comment

Choose a reason for hiding this comment

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

other than the version typo, LGTM

felixfontein
felixfontein reviewed Dec 1, 2025
View reviewed changes
ip2location and others added 5 commits December 2, 2025 06:21
@ip2location @felixfontein
Updated version number.
a5c1e90 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2location_info.py
dca2adc 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2location_info.py
6f4ced8 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location
Updated return definition.
0e69e07
@ip2location
Update BOTMETA.yml to latest module name.
d9a2954
felixfontein
felixfontein reviewed Dec 2, 2025
View reviewed changes
ip2location and others added 2 commits December 3, 2025 06:44
@ip2location @felixfontein
Update plugins/modules/ip2location_info.py
c9dc43c 
Co-authored-by: Felix Fontein <felix@fontein.de>
@ip2location @felixfontein
Update plugins/modules/ip2location_info.py
16949aa 
Co-authored-by: Felix Fontein <felix@fontein.de>
russoz
russoz reviewed Dec 3, 2025
View reviewed changes
plugins/modules/ip2location_info.py Outdated
response, info = fetch_url(
self.module,
url,
force=True,
Copy link
Collaborator

@russoz russoz Dec 3, 2025

Choose a reason for hiding this comment

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

I spotted this in the test case below. Why did you have to use force=True? I don't recall the semantics for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@russoz russoz russoz left review comments

@felixfontein felixfontein felixfontein left review comments

Assignees

No one assigned

Labels

backport-12 Automatically create a backport for the stable-12 branch check-before-release PR will be looked at again shortly before release and merged if possible. merge_commit This PR contains at least one merge commit. Please resolve! module module needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html new_contributor Help guide this first time contributor new_plugin New plugin plugins plugin (any type)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ip2location @ansibullbot @felixfontein @russoz