Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

took out asserts #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

took out asserts #16

wants to merge 1 commit into from

Conversation

wgiddens
Copy link

@wgiddens wgiddens commented Mar 2, 2016

When running on EC2 the AWS_* variables are set implicitly by IAM. They aren't required.

When running on EC2 the AWS_* variables are set implicitly by IAM. They aren't required.
@tzangms
Copy link
Contributor

tzangms commented Mar 20, 2016

Hi @wgiddens

I wan to know more about the AWS_* variables are set implicitly by IAM stuff, could you provide me where the documents is?

I think the asserts should stay, but add some workaround to work on both situation.

@wgiddens
Copy link
Author

You can check the boto3 docs here:
http://boto3.readthedocs.org/en/latest/guide/configuration.html#iam-role

You can set up authentication a variety of ways besides the django settings file, but they recommend letting IAM set it for you.

django-storages catches any errors and raises ImproperlyConfigured. Perhaps that is a solution?

@tzangms
Copy link
Contributor

tzangms commented Mar 28, 2016

@wgiddens Thanks for this information, I'll figure it out and modify for compatibility.

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

Successfully merging this pull request may close these issues.

2 participants