Skip to content

chore(example): Replaced body parser with express JSON parser - #334

Merged
lholmquist merged 1 commit into
cloudevents:mainfrom
deewhyweb:main
Sep 2, 2020
Merged

lholmquist merged 1 commit into
cloudevents:mainfrom
deewhyweb:main

Conversation

@deewhyweb

Copy link
Copy Markdown
Contributor

Proposed Changes

Replaces body parser with standard express body-parse module

Description

  • Fixes Issue #
  • Version:

Signed-off-by: Philip Hayes <phayes@redhat.com>
@lance lance added module/examples Issues related to the examples type/enhancement New feature or request labels Sep 2, 2020

@lance lance left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM - thanks for the contribution @deewhyweb!

@lholmquist lholmquist changed the title Replaced body parser with express JSON parser chore(example): Replaced body parser with express JSON parser Sep 2, 2020
@lholmquist
lholmquist merged commit 4779d89 into cloudevents:main Sep 2, 2020
lholmquist pushed a commit to lholmquist/sdk-javascript that referenced this pull request Sep 8, 2020
…vents#334)

Signed-off-by: Philip Hayes <phayes@redhat.com>

Co-authored-by: Philip Hayes <phayes@redhat.com>
lance pushed a commit that referenced this pull request Sep 9, 2020
Signed-off-by: Philip Hayes <phayes@redhat.com>

Co-authored-by: Philip Hayes <phayes@redhat.com>
lance added a commit to lance/sdk-javascript that referenced this pull request Sep 9, 2020
* chore(example): Replaced body parser with express JSON parser (cloudevents#334)

Signed-off-by: Philip Hayes <phayes@redhat.com>

Co-authored-by: Philip Hayes <phayes@redhat.com>

* fix: upgrade cloudevents from 3.0.1 to 3.1.0 (cloudevents#335)

Snyk has created this PR to upgrade cloudevents from 3.0.1 to 3.1.0.

See this package in npm:
https://www.npmjs.com/package/cloudevents

See this project in Snyk:
https://app.snyk.io/org/lance/project/cb2960b0-db0c-4e77-9ab2-e78efded812e?utm_source=github&utm_medium=upgrade-pr

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Signed-off-by: Lucas Holmquist <lholmqui@redhat.com>

* feat: add a constructor parameter for loose validation (cloudevents#328)

* feat: add a constructor parameter for loose validation

This commit adds a second, optional boolean parameter to the `CloudEvent`
constructor. When `false` is provided, the event constructor will not
perform validation of the event properties, values and extension names.

This commit also modifies the ValidationError class so that the error message
string includes the JSON.stringified version of any schema validation
errors. It also makes the HTTP.toEvent() function create CloudEvent
objects with loose/no validation.

Incorporates comments from cloudevents#328

Fixes: cloudevents#325

Signed-off-by: Lance Ball <lball@redhat.com>

Co-authored-by: Philip Hayes <philip@deewhy.ie>
Co-authored-by: Philip Hayes <phayes@redhat.com>
Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Co-authored-by: Lance Ball <lball@redhat.com>
lance pushed a commit to lance/sdk-javascript that referenced this pull request Sep 9, 2020
…vents#334)

Signed-off-by: Philip Hayes <phayes@redhat.com>

Co-authored-by: Philip Hayes <phayes@redhat.com>
lance pushed a commit that referenced this pull request Sep 11, 2020
Signed-off-by: Philip Hayes <phayes@redhat.com>

Co-authored-by: Philip Hayes <phayes@redhat.com>
@lholmquist lholmquist mentioned this pull request Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module/examples Issues related to the examples type/enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants