Skip to content
This repository has been archived by the owner on Jul 6, 2018. It is now read-only.

http2: add test for invalid pseudo headers #170

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

http2: add test for invalid pseudo headers #170

wants to merge 1 commit into from

Conversation

yosuke-furukawa
Copy link
Member

I add pseudo headers check tests !
Thunks god, it's open source friday!!! https://github.com/blog/2386-contribute-on-open-source-friday

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

http2

sebdeckers

This comment was marked as off-topic.

Sign in to view

mcollina

This comment was marked as off-topic.

Sign in to view

@jasnell
Copy link
Member

jasnell commented Jul 13, 2017

Ping @yosuke-furukawa ... this needs to be rebased :-)

@yosuke-furukawa yosuke-furukawa force-pushed the test_pseudo_header_error branch 2 times, most recently from 8f99d44 to 4388e07 Compare July 15, 2017 02:15
@yosuke-furukawa yosuke-furukawa force-pushed the test_pseudo_header_error branch from 4388e07 to 3476322 Compare July 15, 2017 02:18
@yosuke-furukawa
Copy link
Member Author

sorry for late response, I just fixed. but if it is too late to merge, feel free to ignore this. :)
I would be better to catch up http2 more.

@sebdeckers sebdeckers mentioned this pull request Jul 15, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@addaleax addaleax addaleax left review comments

@sebdeckers sebdeckers sebdeckers left review comments

@mcollina mcollina mcollina left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@yosuke-furukawa @jasnell @mcollina @sebdeckers @addaleax