Skip to content

fix: support mTLS in 1.0 Binary and Structured emitters - #53

Merged
fabiojose merged 3 commits into
cloudevents:masterfrom
lance:48-fix-emitters
Apr 16, 2020
Merged

fabiojose merged 3 commits into
cloudevents:masterfrom
lance:48-fix-emitters

Conversation

@lance

@lance lance commented Apr 8, 2020

Copy link
Copy Markdown
Member

This commit modifies both of the 1.0 emitters so that they may
accept typed objects as a part of the configuration. When using
mTLS in Node, you need to provide an Agent to the underlying
HTTP handler. In this case, Axios will pass this object along to
Node.js when it is provided.

Note: This does not address the issue in 0.3 or below.

Fixes: #48

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

This commit modifies both of the 1.0 emitters so that they may
accept typed objects as a part of the configuration. When using
mTLS in Node, you need to provide an `Agent` to the underlying
HTTP handler. In this case, Axios will pass this object along to
Node.js when it is provided.

Fixes: cloudevents#48

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

Copy link
Copy Markdown
Contributor

@lance, please sign off the commits.

@lance

lance commented Apr 10, 2020 •

Copy link
Copy Markdown
Member Author

Hi @fabiojose. I originally pushed with no signoff. But then amended the commit and force pushed to the branch, but the check didn’t rerun. If you look at the commit, it has been signed off. 3a063d7

Looks like GitHub has caught up

Signed-off-by: Lance Ball <lball@redhat.com>
@lance
lance force-pushed the 48-fix-emitters branch from 0edcad4 to 71a32d4 Compare April 13, 2020 14:11
@fabiojose

Copy link
Copy Markdown
Contributor

@lance, please fill the CHANGELOG.md with those changes and the warning about <0.3 support.

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

lance commented Apr 16, 2020

Copy link
Copy Markdown
Member Author

@fabiojose anything else you are interested in seeing here?

@fabiojose

Copy link
Copy Markdown
Contributor

LGTM

@fabiojose
fabiojose merged commit 02edfb9 into cloudevents:master Apr 16, 2020
@lance
lance deleted the 48-fix-emitters branch April 16, 2020 19:01
@lance lance added the version/1.x.backport Changes that should be backported to the 1.x release line label Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version/1.x.backport Changes that should be backported to the 1.x release line

Projects

None yet

Development

Successfully merging this pull request may close these issues.

getting error using httpsAgent in http binding emitter

2 participants