-
Notifications
You must be signed in to change notification settings - Fork 11
Description
Title
Documentation: Add IPinfoClientCore and IPinfoClientPlus to README
Description
The SDK seems to include fully functional IPinfoClientCore and IPinfoClientPlus clients that are not documented in the README. This caused confusion when trying to access Core tier boolean fields (is_hosting, is_anonymous, etc.).
Current State
The README documents:
IPinfoClient→IPResponseIPinfoClientLite→IPResponseLite
But the SDK also includes (undocumented):
IPinfoClientCore→IPResponseCoreIPinfoClientPlus→IPResponsePlus
The Problem
As a Core tier subscriber, I was using IPinfoClient and couldn't access the boolean fields (IsHosting, IsAnonymous, IsAnycast, IsMobile, IsSatellite) even though the API returns them.
After digging into the source code, I discovered IPinfoClientCore exists and works perfectly:
// This works - but isn't documented
IPinfoClientCore client = new IPinfoClientCore.Builder()
.AccessToken(token)
.Build();
IPResponseCore response = client.IPApi.GetDetails("8.8.8.8");
Console.WriteLine(response.IsHosting); // ✅ works
Console.WriteLine(response.IsAnonymous); // ✅ works
Console.WriteLine(response.Geo.City); // ✅ worksSuggestion
Add a section to the README documenting:
-
IPinfoClientCore for Core tier subscribers
- Endpoint:
api.ipinfo.io/lookup/{ip} - Response model:
IPResponseCore - Available fields including boolean flags
- Endpoint:
-
IPinfoClientPlus for Plus tier subscribers
- Endpoint:
api.ipinfo.io/lookup/{ip} - Response model:
IPResponsePlus - Additional fields like
Abuse,Company,Privacy,Domains
- Endpoint:
-
Client selection guide - which client to use based on subscription tier
Impact
Without this documentation, Core/Plus tier customers may:
- Think the SDK doesn't support their tier's features
- Attempt workarounds like custom HTTP clients
- Miss out on features they're paying for
- Waste a lot of time