Bug#867293: janus-demos: please move janus-demos from contrib to main

[Johannes Schauer]

On Wed, 05 Jul 2017 16:36:15 +0200 Johannes Schauer  wrote:
> the janus-demos package is currently in contrib because the JavaScript it
> ships references scripts which are hosted on cdnjs.cloudflare.com instead of
> versions packaged in Debian.

What must also not be forgotten is to remove all references to:

https://s3.amazonaws.com/github/ribbons/forkme_left_darkblue_121621.png


signature.asc
Description: signature

Bug#867293: janus-demos: please move janus-demos from contrib to main

[Jonas Smedegaard]

Quoting Johannes Schauer (2017-07-05 16:36:15)
> the janus-demos package is currently in contrib because the JavaScript 
> it ships references scripts which are hosted on cdnjs.cloudflare.com 
> instead of versions packaged in Debian. To fix this problem, the 
> respective libraries have to be packaged. What we need is:
> 
> webrtc-adapter: #867274
> sdp: #867272 (needed by webrtc-adapter)
> bootbox: #867286
> blueimp-md5: #867290
> toastr: #867288

Thanks a lot for reporting this, and working on those packages!


> The jQuery BlockUI plugin is also used by the demos but I found that 
> the project seems to suffer from a dead upstream. I posted details in 
> the project's bugtracker: https://github.com/malsup/blockui/issues/141
> 
> Maybe janus upstream can be persuaded to move away from using the 
> blockui library due to its maintenance status.

In the (unlikely!) case that above fail to progress for too long, we 
also have the option of shipping only the non-jQuery demos.


> One last roadblock of moving janus-demos from contrib to main might be 
> the default STUN server used in the JavaScript files: 
> stun:stun.l.google.com:19302
> 
> My intuition is, that a package in main should not by default contact 
> a third party server upon each and every practical usage.

I certainly agree.

Since the Janus developer consider that code only demos, not for 
production use, I expect to simply comment out that line with a patch, 
and mention in README.debian that one or more STUN servers are required 
for those demos to work.

...but please let's discuss that separately: Can I ask you to file a 
separate bugreport about that issue?

 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private


signature.asc
Description: signature

Bug#867293: janus-demos: please move janus-demos from contrib to main

[Johannes Schauer]

Package: janus-demos
Severity: wishlist

Control: block -1 by 867274 867272 867286 867290 867288

Hi,

the janus-demos package is currently in contrib because the JavaScript
it ships references scripts which are hosted on cdnjs.cloudflare.com
instead of versions packaged in Debian. To fix this problem, the
respective libraries have to be packaged. What we need is:

webrtc-adapter: #867274
sdp: #867272 (needed by webrtc-adapter)
bootbox: #867286
blueimp-md5: #867290
toastr: #867288

The jQuery BlockUI plugin is also used by the demos but I found that the
project seems to suffer from a dead upstream. I posted details in the
project's bugtracker: https://github.com/malsup/blockui/issues/141

Maybe janus upstream can be persuaded to move away from using the
blockui library due to its maintenance status.

One last roadblock of moving janus-demos from contrib to main might be
the default STUN server used in the JavaScript files:
stun:stun.l.google.com:19302

My intuition is, that a package in main should not by default contact a
third party server upon each and every practical usage.

Thanks!

cheers, josch