[
https://issues.apache.org/jira/browse/SHINDIG-1772?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13274581#comment-13274581
]
[email protected] commented on SHINDIG-1772:
--------------------------------------------------------
bq. On 2012-05-10 12:38:56, Ryan Baxter wrote:
bq. >
http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/core.io/iotest.js,
line 449
bq. > <https://reviews.apache.org/r/5085/diff/2/?file=108271#file108271line449>
bq. >
bq. > I prefer we use the new constant here
bq.
bq. Erik Bi wrote:
bq. the reason I still see the parameter name instead the constant is
because I want to to keep constant with other code in the makeRequest function,
as you can see, in this function, it checks many parameters, but all use the
string value directly instead of the constant. your opinion?
bq.
bq. Ryan Baxter wrote:
bq. Just because the rest of the code does it doesn't mean its the best
thing to do. At least by using the constants you are "testing" them and making
sure their values are what you expect.
Make sense. I will change it to using constant.
- Erik
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5085/#review7765
-----------------------------------------------------------
On 2012-05-14 12:39:40, Erik Bi wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/5085/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-05-14 12:39:40)
bq.
bq.
bq. Review request for shindig, Ryan Baxter, Dan Dumont, and Stanton Sievers.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. The OpenSocial spec says that you should be able to pass 'SIGN_OWNER' and
'SIGN_VIEWER' as parameters to makeRequest. Shindig does not define these and
in fact looks for the wrong parameters. In io.js Shindig looks for
'OWNER_SIGNED' and 'VIEWER_SIGNED'. In addition 'SIGN_OWNER' and 'SIGN_VIEWER'
should be made constants.
bq.
bq.
bq. This addresses bug shindig-1772.
bq. https://issues.apache.org/jira/browse/shindig-1772
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/core.io/io.js
1327432
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/proxied-form-post/post.js
1327432
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/core.io/iotest.js
1327432
bq.
bq. Diff: https://reviews.apache.org/r/5085/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Update iotest.js
bq.
bq.
bq. Thanks,
bq.
bq. Erik
bq.
bq.
> gadgets.io.RequestParameters.SIGN_OWNER and
> gadgets.io.RequestParameters.SIGN_VIEWER not implemented in Shindig
> ---------------------------------------------------------------------------------------------------------------
>
> Key: SHINDIG-1772
> URL: https://issues.apache.org/jira/browse/SHINDIG-1772
> Project: Shindig
> Issue Type: Bug
> Affects Versions: 2.5.0
> Reporter: Ryan Baxter
> Assignee: Ryan Baxter
>
> The OpenSocial spec says that you should be able to pass 'SIGN_OWNER' and
> 'SIGN_VIEWER' as parameters to makeRequest. Shindig does not define these
> and in fact looks for the wrong parameters. In io.js Shindig looks for
> 'OWNER_SIGNED' and 'VIEWER_SIGNED'. In addition 'SIGN_OWNER' and
> 'SIGN_VIEWER' should be made constants.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
