Skip to content

Support for non-standard ports and cleaner unsecure auth #7

Merged
merged 3 commits into from Oct 18, 2011

2 participants

@adamkaplan

The main change here is support for port in the API host spec. e.g. "baseURL": "localhost:8087", but I also cleaned up the garbage in the URL when no GET params or auth is used.

adamkaplan added some commits Sep 13, 2011
@adamkaplan adamkaplan Removed useless "&undefined=undefined" from URL parameters when no au…
…th key is provided
298c903
@adamkaplan adamkaplan Removed extraneous "?" from URL when no paramteres are supplied 6db92c9
@adamkaplan adamkaplan Added support for API hosts with non-standard ports. Previously an AP…
…I host of "localhost:8080" would actually set "localhost:8080" as the 'host' option to the AJAX request. This would fail horribly. Now the port is split out and passed as the 'port' option to the request. If no ":<port>" is provided, Node will use the protocol default, if any.
5e9f9d4
@mansilladev mansilladev merged commit f2796a6 into mashery:master Oct 18, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.