[ 
https://issues.apache.org/jira/browse/TS-3478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15090438#comment-15090438
 ] 

ASF GitHub Bot commented on TS-3478:
------------------------------------

Github user masaori335 commented on the pull request:

    https://github.com/apache/trafficserver/pull/391#issuecomment-170200715
  
    I'm fine to ship this.
    
    1. Actually I'm not big fun of names of classes and functions in HPACK 
starting with 'HTTP2'. But those confusions are going to be fixed in 
[TS-3967](https://issues.apache.org/jira/browse/TS-3967) with others.
    2. Regression tests worked.


> Indexing header representations on HPACK encoder
> ------------------------------------------------
>
>                 Key: TS-3478
>                 URL: https://issues.apache.org/jira/browse/TS-3478
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: HTTP/2
>            Reporter: Ryo Okubo
>            Assignee: Ryo Okubo
>              Labels: review
>             Fix For: 6.1.0
>
>         Attachments: indexing.patch
>
>
> Support other header field representations on HPACK encoder.
> http://tools.ietf.org/html/draft-ietf-httpbis-header-compression-12#section-6
> Currently the encoder supports only [Literal Header Field never 
> Indexed|http://tools.ietf.org/html/draft-ietf-httpbis-header-compression-12#section-6.2.3].



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to