Empty responses not handled properly #25

Activity

1. indutny commented on Jun 8, 2019

   

   indutny

   on Jun 8, 2019

   Member

   More actions

   👀
2. indutny commented on Jun 9, 2019

   

   indutny

   on Jun 9, 2019

   Member

   More actions

   Work-in-progress fix:

   diff --git a/src/llhttp/http.ts b/src/llhttp/http.ts
   index 969bafc..83779c4 100644
   --- a/src/llhttp/http.ts
   +++ b/src/llhttp/http.ts
   @@ -582,8 +582,8 @@ export class HTTP {
              span.body.end(n('message_done')))));
    
        n('body_identity_eof')
   -      .otherwise(span.body.start(
   -        this.update('finish', FINISH.SAFE_WITH_CB, 'eof')));
   +      .otherwise(
   +        this.update('finish', FINISH.SAFE_WITH_CB, span.body.start(n('eof'))));
    
        // Just read everything until EOF
        n('eof')
   @@ -671,8 +671,9 @@ export class HTTP {
        // Check if we'd like to keep-alive
        if (this.mode === 'strict') {
          n('cleanup')
   -        .otherwise(p.invoke(callback.afterMessageComplete, this.mode === 'strict' ?
   -          { 1: n('restart') } : {}, n('closed')));
   +        .otherwise(p.invoke(callback.afterMessageComplete, {
   +          1: n('restart'),
   +        }, this.update('finish', FINISH.SAFE, n('closed'))));
        } else {
          n('cleanup')
            .otherwise(p.invoke(callback.afterMessageComplete, n('restart')));
   @@ -684,7 +685,7 @@ export class HTTP {
            'Data after `Connection: close`'));
    
        n('restart')
   -      .otherwise(n('start'));
   +      .otherwise(this.update('finish', FINISH.SAFE, n('start')));
      }
    
      private node<T extends Node>(name: string | T): T {

   Writing tests...
3. indutny

   added a commit that references this issue on Jun 9, 2019

   http: fix `llhttp_finish` semantics and add tests

   ...

   This issue will close once commit 4be21df is merged into the 'main' branch.

   4be21df
4. indutny

   closed this as completedin e904107on Jun 10, 2019
5. richardlau

   added a commit that references this issue on Mar 1, 2024

   http: fix `llhttp_finish` semantics and add tests

   ...

   This issue will close once commit e904107 is merged into the 'master' branch.

   e904107