Page MenuHomePhabricator

Add the connection type to the IPInfo API response
Closed, ResolvedPublic3 Estimated Story PointsDec 15 2020

Description

Goal

This task is to add the connection type information to the API response.

@Prtksxna What do we want in the explainer text for "connection type"?

Event Timeline

Niharika triaged this task as Medium priority.Dec 2 2020, 5:47 PM
Niharika created this task.
Restricted Application added a subscriber: Aklapper. ยท View Herald TranscriptDec 2 2020, 5:47 PM
ARamirez_WMF changed the subtype of this task from "Task" to "Deadline".
Niharika changed the point value for this task from 2 to 3.Dec 2 2020, 8:54 PM

I added the to Connection type as an example. I am not sure if that one in particular actually needs it. For the items that are in F33932274, I think:

  • Location doesn't need it
  • ISP / Domain doesn't need it (what do you think?)
  • ASN needs it . From T264837, it could be something like "Autonomous System Number is a collection of connected IP routing prefixes under the control of one or more network operators". I think we should add something here about how it is helpful for patrolling, but that isn't super clear to me. @cwylo, what do you think?
  • Connection type doesn't need it. When you see the data under it (Dialup, Cable/DSL, Corporate, Cellular from T264837) its quite self-explanatory.
  • User type doesn't need it. When you see the data under it (business, cafe, cellular, college, content_delivery_network, dialup, government, hosting, library, military, residential, router, school, search_engine_spider traveler from T264837) its quite self-explanatory.
  • Proxy needs it. It'll also need an explanation of how to use this information while patrolling. The color of the circle is meant to indicato this. For example, tor is bad, residential proxy isn't.
  • Static IP score needs it. From T264837: "An indicator of how static or dynamic an IP address is.".
  • Number of users on this IP doesn't need it.

It is a bit confusing that both Connection Type and User Type share Dialup and Cellular. @Tchanders do they mean different things in these different contexts?

@Prtksxna Sounds good to me. We can user test this and add more explainer texts if needed.

Tchanders renamed this task from Add the connection type to the IP Information box to Add the connection type to the IPInfo API response.Dec 9 2020, 5:29 PM
Tchanders updated the task description. (Show Details)

Change 647802 had a related patch set uploaded (by Tchanders; owner: Tchanders):
[mediawiki/extensions/IPInfo@master] Add ConnectionType to the returned data for an IP address

https://gerrit.wikimedia.org/r/647802

Change 647802 merged by jenkins-bot:
[mediawiki/extensions/IPInfo@master] Add ConnectionType to the returned data for an IP address

https://gerrit.wikimedia.org/r/647802