Skip to content
This repository has been archived by the owner on Jan 20, 2024. It is now read-only.

Customizable soap_cache_file.json location #134

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vsvlasov
Copy link

On read-only file systems (e.g. AWS Lambda) code fails when updates soap_cache_file.json
Instead of raising an exception, that handled by a broad exception and returns the default endpoint URL.

self.update_cache_file(soap_url)
return soap_url
else:
return default_endpoint
except Exception as e:
return default_endpoint

The next query to the default soap endpoint will return 500 Internal server error and without raising an exception debug takes an amount of time and requires deep in FuelSDK code.

Update includes:

  1. Possibility to customize soap_cache_file.json location via environment variable, constructor param, or config file.
  2. PermissionDenied exception will be raised when unable to write soap_cache_file.json

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant