-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Incorrect pack deccoding on EventList. #26
Comments
You should use callback method:
Please, read the documentation and examples on the main page. |
Hello. Code
Output.
My fix of client.py which works for me.
|
I think you need add a event listener, because the events is not part os the response even though they had been generated by the action and announced by the response |
I want make a "helper" to get the events that follows a response with |
No.
It is structure of complex response. And it returns an event list exactly.
AMIClient._next_pack handles responses wrong. It is expected that all
events and responses are ended with a double CLRF what is wrong. There is
no difference what I use, because _next_pack is the only "package
generator".
I think, it may be that ActionCommand is decoded wrong too. I've not tested
it yet.
I've wrote a solution that works, you can test.
Thank you,
Kirill
…On Wed, Nov 21, 2018 at 8:27 PM Éttore Leandro Tognoli < ***@***.***> wrote:
I think you need add a event listener, because the events is not part os
the response even though they had been generated by the action and
announced by the response
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#26 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AMLpAUzbPVj0vXmTcJkIAD4tmQlC67qgks5uxX76gaJpZM4Ynqrh>
.
|
Ettore, no problem.
I've faced a problem with locking client and switched to another library.
I'll make you a patch later on this week.
Thank you,
Kirill
…On Wed, Nov 21, 2018 at 8:35 PM Éttore Leandro Tognoli < ***@***.***> wrote:
I want make a "helper" to get the events that follows a response with EventList:
start but I'm without time these days
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#26 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AMLpAYEP-PaL7xXP4X5XX8ieInK2EeBWks5uxYDJgaJpZM4Ynqrh>
.
|
You have to distinguish between events which are responses to actions, and ones which are unsolicited. The You know how, on the |
Hello.
Unfortunately, AMIClient dispatches responses and events incorrectly.
There are a few types of responses which is marked as EventList. These responses contain a several elements divided by '\r\n\r\n'. As soon, AMIClient wait for this separator to finish pack it cannot catch these types of response.
For example, Action: Status. This action might respond with list of active channels.
But AMIClient passes only first element.
Output is
The text was updated successfully, but these errors were encountered: