Skip to content
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

Multiple attendees can't be added to a CreateMeeting request #7

Closed
yoloVoe opened this issue Jun 21, 2018 · 2 comments
Closed

Multiple attendees can't be added to a CreateMeeting request #7

yoloVoe opened this issue Jun 21, 2018 · 2 comments

Comments

@yoloVoe
Copy link
Contributor

yoloVoe commented Jun 21, 2018

Try making a meeting with multiple participants.
Server will reject your request b/c the generated xml will look like this:

<attendees> <ATTENDEE 1> </attendees> <attendees> <ATTENDEE 2> </attendees>
instead of like this:
<attendees> <ATTENDEE 1> <ATTENDEE 2> </attendees>

The former is rejected by the server, which expects only one <attendees> in the xml payload.

Its b/c of how xml2js works:
Leonidas-from-XIV/node-xml2js#428

I can make a quick PR to fix this if this library still has a maintainer.

@brh55
Copy link
Contributor

brh55 commented Jun 21, 2018

@gk159yo Feel free to submit a PR, we aren't currently using the library in our current projects, but we do use it for any webex related projects.

@yoloVoe
Copy link
Contributor Author

yoloVoe commented Jun 22, 2018

PR: #10

@brh55 brh55 closed this as completed Jun 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants