Serverless Code Version - required

[mindstorms6]

Howdy!

According to the serverless docs : https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#s3-location-object

The version of a code thing isn't required

Data Types
S3 Location Object
Specifies the location of an S3 object as a dictionary containing Bucket, Key, and optional Version properties.

Example:

CodeUri:
  Bucket: mybucket-name
  Key: code.zip
  Version: 121212

But we've got it marked non optional in our code
( i think it's sourced from : https://github.com/awslabs/aws-cdk/blob/4a7eaa6846d54c383022266809e3c3d3100992b6/packages/aws-cdk-resources/cloudformation-specs/000_sam.spec.json#L197 )

And putting a dummy value (like 0 is super not valid)

[rix0rrr]

Yep, that sounds about right.

First reflex: don't use SAM :P

Second reflex: easy fix. Do you need it badly? Also, isn't it about bed time for you?

[eladb]

Can we pull in the latest version of the SAM spec from here

[eladb]

I think this is the latest one https://github.com/awslabs/goformation/blob/master/generate/sam-2016-10-31.json and I think it's aligned to the one we have checked-in.

[rix0rrr]

awslabs/goformation#87

But we get the model from a project called goformation, that's probably not the REAL source of truth, is it?

[rix0rrr]

We'll have to deal with this on a case-by-case basis.