Skip to content
This repository was archived by the owner on May 17, 2019. It is now read-only.

add compatibility with new HttpParser v2.6.2#119

Open
tehrengruber wants to merge 2 commits intoJuliaWeb:masterfrom
tehrengruber:master
Open

add compatibility with new HttpParser v2.6.2#119
tehrengruber wants to merge 2 commits intoJuliaWeb:masterfrom
tehrengruber:master

Conversation

@tehrengruber
Copy link
Copy Markdown

No description provided.

Comment thread src/Requests.jl
ct = split(headers(r)["Content-Type"], ";")[1]
return Nullable(ct)
else
return Nullable{UTF8String}()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Compat.UTF8String

@tehrengruber
Copy link
Copy Markdown
Author

I already saw the usage of Compat.UTF8String in other JuliaWeb packages, but I don't get the point of it to be honest since its just a typealias for String. Any reason to use it?

@yuyichao
Copy link
Copy Markdown
Contributor

0.4 (and possibly 0.3 if applicable) support

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants