In this ShipStationApi constructor:

$this->apiSecret = variable_get('shipstation_api_secret', '');

where as in the shipstation_api_configuration_form:

'#default_value' => variable_get('shipstation_secret', ''),

So, unless the variable is set manually, the API calls will not authenticate.

CommentFileSizeAuthor
#2 shipstation_api-secret_key_variable_name_fix-2989645.patch448 bytesshiraz dindar

Comments

Shiraz Dindar created an issue. See original summary.

shiraz dindar’s picture

shiraz dindar
Location Sooke, BC
commented
Status: Active » Needs review
StatusFileSize
new shipstation_api-secret_key_variable_name_fix-2989645.patch448 bytes

The attached patch renames the constructor var to match the config form var (which is also how it is named in the Ship Station API settings page, ie. without the "key".

andrey.troeglazov’s picture

andrey.troeglazov commented
Assigned: Unassigned » andrey.troeglazov

andrey.troeglazov’s picture

andrey.troeglazov commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.