[dylan-lang/http] 9034d3: Fix bugs in http-client discovered by Piotr Kliber...

  Branch: refs/heads/master
  Home:   https://github.com/dylan-lang/http
  Commit: 9034d398522aa631dc961bc53d37f2e4666d6297
      https://github.com/dylan-lang/http/commit/9034d398522aa631dc961bc53d37f2e4666d6297
  Author: Carl Gay <[email protected]>
  Date:   2014-09-06 (Sat, 06 Sep 2014)

  Changed paths:
    M client/http-client.dylan
    M common/http-common.dylan
    M common/library.dylan
    M documentation/source/reference/common.rst
    M server/core/command-line.dylan
    M server/core/utilities.dylan

  Log Message:
  -----------
  Fix bugs in http-client discovered by Piotr Klibert:

* $default-http[s]-port shouldn't be the same for client and server, so I
  removed it from http-common and instead use unexported constants in server
  and client.  Client gets 80 and 443, server gets 8000 and 8443.  Eventually
  the client shouldn't need these constants since the uri library should supply
  defaults for schemes that specify them.

* Default read-response-body? to #t in http-request(<string>, <request-method>).

* Use parse-url instead of parse-uri in http-request because
  <base-http-request> expects a <url>.  This may not be the correct long-term
  fix though.  Will look into getting rid of the <url> class completely and
  just have <uri>.


  Commit: 36831df30207466833c21eb56425b305c66977ab
      https://github.com/dylan-lang/http/commit/36831df30207466833c21eb56425b305c66977ab
  Author: Bruce Mitchener <[email protected]>
  Date:   2014-09-07 (Sun, 07 Sep 2014)

  Changed paths:
    M client/http-client.dylan
    M common/http-common.dylan
    M common/library.dylan
    M documentation/source/reference/common.rst
    M server/core/command-line.dylan
    M server/core/utilities.dylan

  Log Message:
  -----------
  Merge pull request #71 from cgay/client

Fix bugs in http-client discovered by Piotr Klibert


Compare: https://github.com/dylan-lang/http/compare/53dcef4277c1...36831df30207

_______________________________________________
chatter mailing list
[email protected]
https://lists.opendylan.org/mailman/listinfo/chatter