Handle :scheme and :path in CONNECT

[kornelski]

If the :method is CONNECT the :scheme and :path pseudo-header fields MUST be omitted

This is for #234

[kornelski]

I've verified that Node.js's HTTP/2 can make a request to an h2 server from examples.

const http2 = require('http2');

const client = http2.connect('http://127.0.0.1:5928');

// Must not specify the ':path' and ':scheme' headers
// for CONNECT requests or an error will be thrown.
const req = client.request({
  ':method': 'CONNECT',
  ':authority': `127.0.0.1:5928`,
});

req.on('response', (headers) => {
  console.log("resp", headers[http2.constants.HTTP2_HEADER_STATUS], headers);
});

req.on('error', (err) => console.error("boo", err));
req.setEncoding('utf8');
let data = '';
req.on('data', (chunk) => data += chunk);
req.on('end', () => {
  console.log(`The server says: ${data}`);
  client.close();
});
req.end('Jane');

[seanmonstar]

Looks right, thanks!

We should probably add a test in h2-tests/tests/server.rs exercising this.

[kornelski]

I've added a test and rebased on master.