Display API failure errors in the IPInfo popup
Closed, ResolvedPublic3 Estimated Story Points
Actions

Assigned To

Authored By

	Tchanders
	Sep 21 2020, 11:51 AM

Description

After T260604: Display a popup with fixed mock data for an IP address on the specified page(s), mock data is displayed in the IPInfo popup, when the API successfully returns data.

When the API responds with a failure code, we should also display an appropriate failure message.

Questions:

Note that T263409 is specifically for displaying timeout errors.

Popup error states:

Figma

Mentioned In: T271675: Create loading and error states for the IP info box and popup [M]
T271670: Figure out what the IP info box should show when some information is not available
T260604: Display a popup with fixed mock data for an IP address on the specified page(s)
Mentioned Here: T260603: Create an API endpoint that accepts a log id or revision id and returns mock data about the IP address
T264879: IPInfo: Consider permissions logic for getting IP information from log entries
T299050: Remove access to the IPInfo API if the beta feature is not enabled [M]
T267402: IPInfo: Blocked users can use IPInfo
T260604: Display a popup with fixed mock data for an IP address on the specified page(s)
T263409: Display pending element and error message on the IPInfo popup, where appropriate

@Prtksxna Adding this to your column. Let's talk more about this in our meeting.

Here are some of the current error messages for logs and revisions and the circumstances when they appear. I am not sure it is exhaustive:

Case	HTTP code	Log Message	Revision Message
Logged out	401	"You do not have permission to perform the action"	"You do not have permission to perform the action"
User pref disabled or no `ipinfo` right	403	"You do not have permission to perform the action"	"You do not have permission to perform the action"
Log or revision does not exist	404	"Log entry does not exist"	"The specified revision (<rev id>) does not exist"
User does not have permission to log/revision	403	"Your account does not have access to view the performer of the log entry or the log entry itself"	"Your account does not have access to view the author of the revision"
Log/revision is not an IP	404	"Cannot retrieve ipinfo for logs of registered users"	"Cannot retrieve ipinfo for revisions of registered users"
No error, but API returns no information	200	"There is no data for this IP address from GeoIp2"	"There is no data for this IP address from GeoIp2"

Noting that T267402: IPInfo: Blocked users can use IPInfo introduces another kind of error.

STran set the point value for this task to 3.

Duplicate (see tickets above)

	F34770067: image.png
	Nov 26 2021, 4:00 PM