scaleway / serverless-scaleway-functions

Plugin for Serverless Framework to allow users to deploy their serverless applications on Scaleway Functions
MIT License
78 stars 25 forks source link

set timeout in seconds in container examples #145

Closed alelapeyre closed 1 year ago

alelapeyre commented 1 year ago

Hello!

I got the following error when using the container example with the timeout value set:

→ sls deploy                                                                                                                                                                                         
...
Environment: linux, node 19.7.0, framework 3.16.0, plugin 6.2.2, SDK 4.3.2
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issue

Error:
Error: proto: syntax error (line 1:183): unexpected token 20000
    at manageError (<path>/node_modules/serverless-scaleway-functions/shared/api/utils.js:23:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Promise.all (index 0)

Searching a bit, I found that the timeout value should be provided in seconds. So here is a small PR to update examples containing the timeout field with a valid value.

thomas-tacquet commented 1 year ago

Hello 👋

Cool nice find thank your for your contribution we really appreciate it👌