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.

Sign up for GitHub

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

Aws cloudwatchlog event schema #8228

Merged
merged 8 commits into from Sep 22, 2020
Merged

Aws cloudwatchlog event schema #8228

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d228250
awsLogGroup regex schema definition
Sep 14, 2020
0d91ae8
aws cloudwatchlog event schema
Sep 14, 2020
bba26f1
fix schema
Sep 14, 2020
8924fc0
remove comment
Sep 14, 2020
e96ea5a
update awsLogGroupName regex
Sep 15, 2020
9304bbd
fix name regex
thewizarodofoz Sep 15, 2020
04d6e4d
fix aws log group name regex
Sep 22, 2020
8eb7169
remove inline validations and validations tests
Sep 22, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
14 changes: 12 additions & 2 deletions lib/plugins/aws/package/compile/events/cloudWatchLog/index.js
View file
Open in desktop
Expand Up @@ -11,9 +11,19 @@ class AwsCompileCloudWatchLogEvents {
'package:compileEvents': this.compileCloudWatchLogEvents.bind(this),
};

// TODO: Complete schema, see https://github.com/serverless/serverless/issues/8027
this.serverless.configSchemaHandler.defineFunctionEvent('aws', 'cloudwatchLog', {
anyOf: [{ type: 'string' }, { type: 'object' }],
anyOf: [
{ $ref: '#/definitions/awsLogGroupName' },
{
type: 'object',
properties: {
logGroup: { $ref: '#/definitions/awsLogGroupName' },
filter: { type: 'string' },
},
required: ['logGroup'],
additionalProperties: false,
},
],
});
}

Expand Down
4 changes: 4 additions & 0 deletions lib/plugins/aws/provider/awsProvider.js
View file
Open in desktop
Expand Up @@ -233,6 +233,10 @@ class AwsProvider {
UpdateReplacePolicy: { type: 'string' },
Condition: { type: 'string' },
},
awsLogGroupName: {
type: 'string',
pattern: '[.-_/#A-Za-z0-9]+',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe we should either escape - or put it as last character, as otherwise it'll expand to range between . and _ chars.

Also regex should be enclosed into ^ and $, as otherwise we confirm that any part in strings matches and not string as a whole

},
},
provider: {
properties: {
Expand Down