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

add 'scheme' on ql-http:url. 'cl-http:fetch' support adding handler f… #124

Merged
merged 4 commits into from Jan 20, 2016

Conversation

@snmsts
Copy link
Contributor

commented Dec 1, 2015

…or each scheme.

I think with this patch https protocol are easily supported with small patch.(using external library or software)

@snmsts snmsts force-pushed the snmsts:support-scheme branch from 360c856 to 1bd64b5 Jan 19, 2016
@snmsts

This comment has been minimized.

Copy link
Contributor Author

commented Jan 19, 2016

It's my fault to use 'git rebase' and lost the comment.. but I think I've done with the comments for the code.

quicklisp added a commit that referenced this pull request Jan 20, 2016
add 'scheme' on ql-http:url. 'cl-http:fetch' support adding handler f…
@quicklisp quicklisp merged commit e5ff41e into quicklisp:master Jan 20, 2016
@snmsts snmsts deleted the snmsts:support-scheme branch Jan 21, 2016
@quicklisp

This comment has been minimized.

Copy link
Owner

commented Feb 1, 2016

The change of the default port value from 80 to NIL causes problems with requests. See #129 for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.