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

Adds basic Requests library #44

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

Adds basic Requests library #44

wants to merge 6 commits into from

Conversation

chuckha
Copy link
Member

@chuckha chuckha commented Mar 27, 2013

With a fixed history.

@despeset
Copy link
Member

  • requests.jl should probably be src/Requests.jl
  • HttpMethodBitmaskToName(method) on line 27 needs to be HttpMethodBitmaskToName[method]
  • request("http://google.com") seems to return the DNS response, it should be followed.
  • request("http://www.google.com") fails with the below error – see the Julia issue about getaddrinfo
ERROR: getaddrinfo callback: no such file or directory (ENOENT)
 in _uv_hook_getaddrinfo at socket.jl:201
 in process_events at stream.jl:312
 in event_loop at multi.jl:1381
 in anonymous at client.jl:284

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.

2 participants