Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update request_header.c #14

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update request_header.c #14

wants to merge 5 commits into from

Conversation

Danath
Copy link

@Danath Danath commented Apr 1, 2015

a correct expression is given to the request end of RequestLineEnd.

a correct expression is given to the request end of RequestLineEnd.
@Danath Danath closed this Apr 1, 2015
@Danath Danath reopened this Apr 1, 2015
@c9s
Copy link
Owner

c9s commented Jul 29, 2015

Hi @Danath ,

Can you remove the Makefile commit? we're using cmake so that the Makefile you provided will be overwritten by the auto-generated one.

@Danath
Copy link
Author

Danath commented Jul 29, 2015

Sorry ! I was not on purpose for this commit, Just pay no attention to the makefile

@litesky
Copy link

litesky commented Nov 14, 2015

add header->RequestLineEnd
At request_header.c line 130
Source:

    // end of header
    if (iscrlf(p)) return 0;

Destination:

    // end of header
    if (iscrlf(p)) {
      header->RequestLineEnd= p+2;
      return 0;
    };

@c9s
Copy link
Owner

c9s commented Nov 14, 2015

Please delete the libhttp.dll from the commit XD

@xtfpg
Copy link

xtfpg commented Nov 29, 2016

High Level API & Low Level API can not to use?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants