macielbombonato / bigbluebutton

Automatically exported from code.google.com/p/bigbluebutton
0 stars 0 forks source link

Malformed recordings XML #912

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
I don't know if this is the case in all recordings, but the sample recording 
Richard gave me to work with has a flaw. It seems that the <slidesInfo> event 
contents are parsed as an HTML safe string, when the contents should be parsed 
as an XML structure. This is what my <slidesInfo> xml looks like:

<slidesInfo>&lt;uploadedpresentation&gt;
  &lt;conference id='cda73e7e-8f71-4906-8e74-cbdc66a5e97f' room='cda73e7e-8f71-4906-8e74-cbdc66a5e97f'&gt;
    &lt;presentation name='Building-a-Scalable-Architecture'&gt;
      &lt;slides count='46'&gt;
        &lt;slide number='1' name='slide/1' thumb='thumbnail/1' /&gt;
        &lt;slide number='2' name='slide/2' thumb='thumbnail/2' /&gt;
        &lt;slide number='3' name='slide/3' thumb='thumbnail/3' /&gt;
        &lt;slide number='4' name='slide/4' thumb='thumbnail/4' /&gt;
        &lt;slide number='5' name='slide/5' thumb='thumbnail/5' /&gt;
        &lt;slide number='6' name='slide/6' thumb='thumbnail/6' /&gt;
        &lt;slide number='7' name='slide/7' thumb='thumbnail/7' /&gt;
        &lt;slide number='8' name='slide/8' thumb='thumbnail/8' /&gt;
        &lt;slide number='9' name='slide/9' thumb='thumbnail/9' /&gt;
        &lt;slide number='10' name='slide/10' thumb='thumbnail/10' /&gt;
        &lt;slide number='11' name='slide/11' thumb='thumbnail/11' /&gt;
        &lt;slide number='12' name='slide/12' thumb='thumbnail/12' /&gt;
        &lt;slide number='13' name='slide/13' thumb='thumbnail/13' /&gt;
        &lt;slide number='14' name='slide/14' thumb='thumbnail/14' /&gt;
        &lt;slide number='15' name='slide/15' thumb='thumbnail/15' /&gt;
        &lt;slide number='16' name='slide/16' thumb='thumbnail/16' /&gt;
        &lt;slide number='17' name='slide/17' thumb='thumbnail/17' /&gt;
        &lt;slide number='18' name='slide/18' thumb='thumbnail/18' /&gt;
        &lt;slide number='19' name='slide/19' thumb='thumbnail/19' /&gt;
        &lt;slide number='20' name='slide/20' thumb='thumbnail/20' /&gt;
        &lt;slide number='21' name='slide/21' thumb='thumbnail/21' /&gt;
        &lt;slide number='22' name='slide/22' thumb='thumbnail/22' /&gt;
        &lt;slide number='23' name='slide/23' thumb='thumbnail/23' /&gt;
        &lt;slide number='24' name='slide/24' thumb='thumbnail/24' /&gt;
        &lt;slide number='25' name='slide/25' thumb='thumbnail/25' /&gt;
        &lt;slide number='26' name='slide/26' thumb='thumbnail/26' /&gt;
        &lt;slide number='27' name='slide/27' thumb='thumbnail/27' /&gt;
        &lt;slide number='28' name='slide/28' thumb='thumbnail/28' /&gt;
        &lt;slide number='29' name='slide/29' thumb='thumbnail/29' /&gt;
        &lt;slide number='30' name='slide/30' thumb='thumbnail/30' /&gt;
        &lt;slide number='31' name='slide/31' thumb='thumbnail/31' /&gt;
        &lt;slide number='32' name='slide/32' thumb='thumbnail/32' /&gt;
        &lt;slide number='33' name='slide/33' thumb='thumbnail/33' /&gt;
        &lt;slide number='34' name='slide/34' thumb='thumbnail/34' /&gt;
        &lt;slide number='35' name='slide/35' thumb='thumbnail/35' /&gt;
        &lt;slide number='36' name='slide/36' thumb='thumbnail/36' /&gt;
        &lt;slide number='37' name='slide/37' thumb='thumbnail/37' /&gt;
        &lt;slide number='38' name='slide/38' thumb='thumbnail/38' /&gt;
        &lt;slide number='39' name='slide/39' thumb='thumbnail/39' /&gt;
        &lt;slide number='40' name='slide/40' thumb='thumbnail/40' /&gt;
        &lt;slide number='41' name='slide/41' thumb='thumbnail/41' /&gt;
        &lt;slide number='42' name='slide/42' thumb='thumbnail/42' /&gt;
        &lt;slide number='43' name='slide/43' thumb='thumbnail/43' /&gt;
        &lt;slide number='44' name='slide/44' thumb='thumbnail/44' /&gt;
        &lt;slide number='45' name='slide/45' thumb='thumbnail/45' /&gt;
        &lt;slide number='46' name='slide/46' thumb='thumbnail/46' /&gt;
      &lt;/slides&gt;
    &lt;/presentation&gt;
  &lt;/conference&gt;
&lt;/uploadedpresentation&gt;</slidesInfo>

Instead, all those &lt and &gt should be < and >

Original issue reported on code.google.com by Me.S...@gmail.com on 31 Mar 2011 at 11:13

GoogleCodeExporter commented 8 years ago
yeah... today, I noticed, too... that's because in bbb-web, it's generated an 
xml with the info about slides, then the xml is passed to bbb-apps, but in 
there is never parsed , it's used as it comes... In theory, I did a method for 
join that xml with the recording xml... I'll take a look...

Original comment by mcmarkos86 on 31 Mar 2011 at 11:23

GoogleCodeExporter commented 8 years ago
Also, there's a lot of unnecessary nesting in that slidesInfo xml. There's 5 
levels of nesting without any useful info in between:
        </slides>
      </presentation>
    </conference>
  </uploadedpresentation>
</slidesInfo>

Maybe this can be improved a bit.

Original comment by Me.S...@gmail.com on 31 Mar 2011 at 11:26

GoogleCodeExporter commented 8 years ago
I agree...

Original comment by mcmarkos86 on 31 Mar 2011 at 11:27

GoogleCodeExporter commented 8 years ago
Thanks Markos

Original comment by Me.S...@gmail.com on 31 Mar 2011 at 11:30

GoogleCodeExporter commented 8 years ago

Original comment by mcmarkos86 on 23 Nov 2011 at 12:05