SemiConscious commented on issue #1039:
URL: https://github.com/apache/royale-asjs/issues/1039#issuecomment-755300613


   @safrizal @carlosrovira @greg-dove - First, I'm really sorry if this change 
created problems for you. Next: the problem is that the existing code contains 
bugs - royale AMF (in it's currently reverted state) cannot run the same code 
as it did under flex because (to cut to the chase) the `NetConnection.send()` 
function has a significantly different interface - `params` is a single 
parameter instead of a `...` parameter as it used to be. The question then 
becomes: is it possible to run Flex-style AMF applications using the new 
interface? The answer as far as I can see is no, even if you refactor the 
parameter passing. If it was a matter of refactoring I would have refactored.
   
   I posed this question in my pull request: why has the interface changed? If 
we could answer that question that would be a good start.
   
   Second - @safrizal how on earth are you getting your AMF services to work 
without using the target URI, which is currently not being passed to the AMF 
service at all? Are you just using the failover service hook for when there is 
no URI? If so ok but that shouldn't mean other people cannot use it in the way 
that was originally intended.
   
   Happy to work on this more with anyone interested.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to