Skip to content

fix: HTTP headers for extensions with false values - #493

Merged
lance merged 2 commits into
cloudevents:mainfrom
lance:lance-492-false-extension-value
Jun 14, 2022
Merged

lance merged 2 commits into
cloudevents:mainfrom
lance:lance-492-false-extension-value

Conversation

@lance

@lance lance commented Jun 14, 2022

Copy link
Copy Markdown
Member

CloudEvent objects may include extensions that have a defined key and a
false value. This change ensures that HTTP messages for CloudEvents
containing these extension values include the appropriate headers.

Fixes: #492

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

CloudEvent objects may include extensions that have a defined key and a
`false` value. This change ensures that HTTP messages for CloudEvents
containing these extension values include the appropriate headers.

Signed-off-by: Lance Ball <lball@redhat.com>
@lance
lance requested a review from a team June 14, 2022 19:02
@lance lance self-assigned this Jun 14, 2022
@lance lance added the type/fix A change that fixes something that is broken label Jun 14, 2022
Signed-off-by: Lance Ball <lball@redhat.com>
@lance

lance commented Jun 14, 2022

Copy link
Copy Markdown
Member Author

Closing and reopening to re-trigger CI with updated package.json

@lance lance closed this Jun 14, 2022
@lance lance reopened this Jun 14, 2022
@lance
lance merged commit d6f52ca into cloudevents:main Jun 14, 2022
@lance
lance deleted the lance-492-false-extension-value branch June 14, 2022 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/fix A change that fixes something that is broken

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Binary mode header missing extension attribute when the value is false (boolean)

2 participants