★ wanayoo — archive 1999 https://github.com/nficano/python-lambda/issues/168Nouvelle recherche | Portail wanayoo
Skip to content

/ python-lambda

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

Jump to bottom

TypeError: str expected, not int #168

Open
aluciffer opened this issue Aug 9, 2019 · 2 comments
Open

TypeError: str expected, not int #168

aluciffer opened this issue Aug 9, 2019 · 2 comments

Comments

@aluciffer
Copy link

@aluciffer aluciffer commented Aug 9, 2019 •
edited

Hi,

I get this 'str expected, not int' error that gets traced back to :

File "...\aws_lambda\aws_lambda.py", line 217, in invoke
    os.environ[key] = get_environment_variable_value(value)

This is because i have configured an int variable in my config.yaml, i.e.:
repeat: 100

Would it be good to wrap get_environment_variable_value in str() ?
@urandu
Copy link

@urandu urandu commented Oct 20, 2019

I think It would be good to wrap it in a str() like its done at this point

python-lambda/aws_lambda/aws_lambda.py

Line 662 in aad2dad

key: str(get_environment_variable_value(value))

urandu added a commit to urandu/python-lambda that referenced this issue Oct 20, 2019
@urandu
cast get_environment_variable_value output to str
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
11db872 
this addresses this issue nficano#168
@urandu urandu mentioned this issue Oct 20, 2019
Open
@urandu
Copy link

@urandu urandu commented Oct 20, 2019

Here is the PR #270
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@aluciffer @urandu
You can’t perform that action at this time.