[jitsi-dev] [jitsi/jitsi-videobridge] Return error details (#294)


#1

The purpose if this PR is to make JVB return meaningful error when usually it would have returned "feature-not-implemented" error.
You can view, comment on, or merge this pull request online at:

  https://github.com/jitsi/jitsi-videobridge/pull/294

-- Commit Summary --

  * refactor(Videobridge): use IQUtils.createError
  * refactor(Content): remove deprecated getSctpConnection(Endpoint)
  * refactor(Videobridge): return error details on Conference access/create
  * style(Videobridge): decrease indentation after previous commit
  * refactor(Videobridge): return error when fails to allocate new Content
  * style(Videobridge): decrease indentation after previous commit
  * refactor(Videobridge): return error on Channel create/access/expire
  * style(Videobridge): decrease indentation after previous commit
  * refactor(Videobridge): return error on SCTP connection access/expire
  * fix(ComponentImpl): return internal server error when no Videobridge
  * fix(ComponentImpl): return error when fails to convert to Smack IQ

-- File Changes --

    M pom.xml (2)
    M src/main/java/org/jitsi/videobridge/Content.java (16)
    M src/main/java/org/jitsi/videobridge/Videobridge.java (725)
    M src/main/java/org/jitsi/videobridge/xmpp/ComponentImpl.java (26)
    M src/test/java/org/jitsi/videobridge/FocusControlTest.java (9)

-- Patch Links --

https://github.com/jitsi/jitsi-videobridge/pull/294.patch
https://github.com/jitsi/jitsi-videobridge/pull/294.diff

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/294


#2

@bgrozev pushed 3 commits.

114cedf refactor: Simplify an if condition a bit.
7040737 refactor: Moves code that sets properties on a Channel
53cfadf fix: Don't throw an error if a channel

···


You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/294/files/3583c46a13ec28538a7cd92811e5839de0313226..53cfadf031a412f94b83f283d31bd22a0c7d3264


#3

Merged #294.

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/294#event-766397897