{"comment":{"url":"https://api.github.com/repos/mitmproxy/netlib/comments/11570371","html_url":"https://github.com/mitmproxy/netlib/commit/fdbb3b76cf8cd7caaa644dc31e48521096ed5349#commitcomment-11570371","id":11570371,"user":{"login":"Kriechi","id":114300,"avatar_url":"https://avatars.githubusercontent.com/u/114300?v=3","gravatar_id":"","url":"https://api.github.com/users/Kriechi","html_url":"https://github.com/Kriechi","followers_url":"https://api.github.com/users/Kriechi/followers","following_url":"https://api.github.com/users/Kriechi/following{/other_user}","gists_url":"https://api.github.com/users/Kriechi/gists{/gist_id}","starred_url":"https://api.github.com/users/Kriechi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Kriechi/subscriptions","organizations_url":"https://api.github.com/users/Kriechi/orgs","repos_url":"https://api.github.com/users/Kriechi/repos","events_url":"https://api.github.com/users/Kriechi/events{/privacy}","received_events_url":"https://api.github.com/users/Kriechi/received_events","type":"User","site_admin":false},"position":9,"line":83,"path":"netlib/http2/frame.py","commit_id":"fdbb3b76cf8cd7caaa644dc31e48521096ed5349","created_at":"2015-06-08T15:56:26Z","updated_at":"2015-06-08T15:56:26Z","body":"The reason why I added this is, that I was tracking down a test failure in pathod - which does not yet support HTTP/2. After figuring that out I decided to print some kind of warning...\r\nI think the more suitable approach would be to deny any requests if TLS & ALPN failed.\r\nCurrently ALPN (or even TLS) is NOT enforced - which is probably against the RFC."}}