When using the Amazon Store Item Detail Page panel rather than the default /amazon_store/item/{ASIN} handler (as outlined in the module documentation, under the heading Enabling and configuring the Item Detail Panel):

If amazon_store_amazon_item_context() receives an error response from the amazon_store_retrieve_item() function, the user will see the site's 404 (page not found) handler page.

See issues #954992: Amazon Store produces 404 Page Not Found when navigating to amazon_store/item/ASIN item page and related #1170144: InvalidParameterValue on legitimate B0015EWMX8 ASIN (but Amazon Store might work OK) for background information and steps to reproduce.

Suggested solutions

I think it would be better if the panel returned a formatted, and much friendlier, error message as the panel's HTML.

At a minimum, the default message would indicate that there is a temporary problem with the Amazon system, and to try again later.

A better solution might be to display other store products along with the error message. So, I think we might need a theme function or other generalized function to format an error message given some contextual information.

Comments

mcurry’s picture

mcurry commented
Issue tags: +'page not found' errors, +error handling

Updating issue tags

mcurry’s picture

mcurry commented
Title: Amazon Store Item Detail Page panel page (page-amazon_item) generates 404 page on amazon_store_retrieve_item() error response » Amazon Store Item Detail Page panel page generates 404 page on error response from Amazon Servers
Assigned: Unassigned » mcurry

This problem can be triggered by spurious and transient errors from the Amazon servers. Therefore, I think we need to be better at error handling in the Amazon Store application side (the Amazon Module API is fine, as is this module's low-level API code).