Skip to content

Fix crash on H3 transaction#9786

Merged
maskit merged 1 commit intoapache:masterfrom
maskit:h3_fix_closing_event
Jun 8, 2023
Merged

Fix crash on H3 transaction#9786
maskit merged 1 commit intoapache:masterfrom
maskit:h3_fix_closing_event

Conversation

@maskit
Copy link
Copy Markdown
Member

@maskit maskit commented Jun 6, 2023

The functions can be called without an event if the event handler of H3Transaction is called by H3App.

This closes #9782

@maskit maskit added the HTTP/3 label Jun 6, 2023
@maskit maskit added this to the 10.0.0 milestone Jun 6, 2023
@maskit maskit self-assigned this Jun 6, 2023
@bryancall
Copy link
Copy Markdown
Contributor

[approve ci autest]

1 similar comment
@brbzull0
Copy link
Copy Markdown
Contributor

brbzull0 commented Jun 7, 2023

[approve ci autest]

Copy link
Copy Markdown
Contributor

@brbzull0 brbzull0 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks.

@maskit
Copy link
Copy Markdown
Member Author

maskit commented Jun 7, 2023

[approve ci autest]

@maskit maskit merged commit 5bf67d6 into apache:master Jun 8, 2023
bryancall pushed a commit to bryancall/trafficserver that referenced this pull request Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Http3 transaction crash

3 participants