[jitsi-dev] [jitsi-videobridge] Removed path from recording result (#32)


#1

As discussed on the mailing list, the full path discloses information about the storage architecture. I've replaced the path with the directory name. That allows an application that knows the storage architecture to access the recording, without knowing the internal implementation of Jitsi. At first I wanted to remove it completely, but then applications would have to know how jitsi generates the directory name.
You can merge this Pull Request by running:

  git pull https://github.com/nwittstruck/jitsi-videobridge recording-directory

Or you can view, comment on it, or merge it online at:

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

-- Commit Summary --

  * removed path from recording result

-- File Changes --

    M lib/protocol-jabber.jar (0)
    M src/org/jitsi/videobridge/Conference.java (31)
    M src/org/jitsi/videobridge/Videobridge.java (4)
    M src/org/jitsi/videobridge/rest/JSONDeserializer.java (8)
    M src/org/jitsi/videobridge/rest/JSONSerializer.java (8)

-- Patch Links --

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

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/32


#2

Merged in cd9ae92a277630259643effbac590cbfb23d3bdc
Thanks!

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/32#issuecomment-61940134


#3

Closed #32.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/32#event-189163579