Add IP Risk and Anonymous Plus inputs to Insights #279
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
oschwald
commented
Nov 6, 2025
oschwald
commented
- Add CLAUDE.md documentation for AI assistants
- Add anonymizer and IP risk data to Insights web service
This adds guidance for Claude Code and other AI assistants when working with the codebase, including architecture patterns, testing conventions, and development workflows. Also update .gitignore to exclude Claude Code directories. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
oschwald
force-pushed
the
greg/eng-3314-ip-risk-and-anonymous-plus-outputs-are-supported-by-geoip2
branch
from
ee46b69 to
b2d7444
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR adds anonymizer data and IP risk scoring to the GeoIP2 Insights model. It introduces a new Anonymizer record class and deprecates the corresponding properties in the Traits class in favor of the new dedicated anonymizer object.
- Adds a new
Anonymizerrecord class with VPN and anonymity detection properties - Adds
ipRiskSnapshotproperty to theTraitsclass for static risk scoring - Deprecates anonymity-related boolean properties in
Traitsin favor of the newanonymizerobject
Reviewed Changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/Record/Anonymizer.php | New record class containing anonymizer data including confidence scores, VPN provider information, and various anonymity flags |
| src/Model/Insights.php | Extended to include an anonymizer property and properly serialize it |
| src/Record/Traits.php | Added ipRiskSnapshot property and deprecated six anonymity-related properties in favor of the anonymizer object |
| tests/GeoIp2/Test/Model/InsightsTest.php | Comprehensive test coverage for the new anonymizer property and ipRiskSnapshot field |
| CHANGELOG.md | Documents the new features and deprecations for version 3.3.0 |
| CLAUDE.md | New documentation file providing guidance for Claude Code when working with this repository |
| .gitignore | Added exclusions for Claude Code directories |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This commit adds support for the new anonymizer object and IP risk snapshot field in the GeoIP2 Insights web service response. Key changes: - Add new Anonymizer record class with VPN detection fields (confidence, provider name, network last seen, and anonymity flags) - Add anonymizer property to Insights model - Add ipRiskSnapshot field to Traits record for static IP risk scoring - Deprecate anonymous IP flags in Traits (isAnonymous, isAnonymousVpn, isHostingProvider, isPublicProxy, isResidentialProxy, isTorExitNode) in favor of the new anonymizer object - Update tests to cover new fields - Update CHANGELOG for version 3.3.0 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
oschwald
force-pushed
the
greg/eng-3314-ip-risk-and-anonymous-plus-outputs-are-supported-by-geoip2
branch
from
b2d7444 to
341119e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.