Simplify the locker core proxying #11
Comments
|
Unicode chars get mashed in the process, this should also be fixed. This can become a second issue if needed. |
smurthas
added a commit
that referenced
this issue
Apr 12, 2011
Ops/webservice.js: merged proxied and proxiedPost, fixed unicode issues tests/connector-twitter-test.js: small updates to reflect resolution of proxy issues tests/locker-core-api-test.js: added better error info for unicode issues
|
The proxy should also respond to and propagate redirects. |
smurthas
added a commit
that referenced
this issue
Apr 13, 2011
Ops/webservice.js: proxy redirect fixes tests/Me/proxy-redirect-test/: webservice for testing redirects tests/locker-issues-test.js: tests for proxy redirect Connectors/Twitter/client.js: Small bug fixes to error states
|
This is resolved as of the referenced commit. Closing. |
temas
pushed a commit
that referenced
this issue
Apr 14, 2011
* Common/node/locker.js
- Remove the id from a few more function calls
* Connectors/Facebook/client.js
- Updates to the new locker client interfaces
* Connectors/Flickr/client.js
- Updates to the new locker client interfaces
* Connectors/Twitter/client.js
- Updates to the new locker client interfaces
* Connectors/foursquare/client.js
- Updates to the new locker client interfaces
* tests/connector-flickr-test.js
- Updates to the new locker client interfaces
* tests/locker-client-tests.js
- Added the event test
All tests pass except for the issue #11 test
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The proxy functions are mostly mirrored in the GET and POST handlers as well as the proxied() and proxiedPost(). Merge this up. Clean this up. Win.
The text was updated successfully, but these errors were encountered: