Jump to content


Photo

OpenWebif [E2OpenPlugins]


  • Please log in to reply
1459 replies to this topic

Re: OpenWebif [E2OpenPlugins] #421 andyblac

  • Senior Member
  • 465 posts

+18
Neutral

Posted 28 December 2011 - 18:56

timer list not very neat.

Attached Files



Re: OpenWebif [E2OpenPlugins] #422 Rincewind

  • Member
  • 25 posts

+12
Neutral

Posted 28 December 2011 - 19:45

timer list not very neat.


I have implemented a fix for this, but I don't know how to create the git patch from my repository with TortoiseGIT.
Could anybody tell me how to do that?

Re: OpenWebif [E2OpenPlugins] #423 blzr

  • PLi® Core member
  • 2,270 posts

+118
Excellent

Posted 28 December 2011 - 19:50

Hmm, i tried streaming with new webif and it works but firefox doesnt want to automatically start VLC. I get confirmation window with stream.m3u.txt file...
With original webinf VLC opens automatically.
...


same problem here after today's update (and it worked as it should on Monday)
it prompts for confirmation every time...

Attached Files


True sarcasm doesn't need green font...

Re: OpenWebif [E2OpenPlugins] #424 nobody9

  • Senior Member
  • 187 posts

+76
Good

Posted 28 December 2011 - 19:57

will be fixed later today,
we must change the setting of the conten-type, at the moment will always overriten with text/plain



Hmm, i tried streaming with new webif and it works but firefox doesnt want to automatically start VLC. I get confirmation window with stream.m3u.txt file...
With original webinf VLC opens automatically.
...


same problem here after today's update (and it worked as it should on Monday)
it prompts for confirmation every time...



Re: OpenWebif [E2OpenPlugins] #425 Rincewind

  • Member
  • 25 posts

+12
Neutral

Posted 28 December 2011 - 20:03


timer list not very neat.


I have implemented a fix for this, but I don't know how to create the git patch from my repository with TortoiseGIT.
Could anybody tell me how to do that?


OK, Google (possibly) helped...
Here is the patch for the scrambled timer list: if something is wrong in the packaging, please let me know.

Attached Files


Edited by Rincewind, 28 December 2011 - 20:03.


Re: OpenWebif [E2OpenPlugins] #426 nobody9

  • Senior Member
  • 187 posts

+76
Good

Posted 28 December 2011 - 20:04

Context menu

TortoiseGit->Diff with previous version

i only see some screen-shots in the net


timer list not very neat.


I have implemented a fix for this, but I don't know how to create the git patch from my repository with TortoiseGIT.
Could anybody tell me how to do that?



Re: OpenWebif [E2OpenPlugins] #427 nobody9

  • Senior Member
  • 187 posts

+76
Good

Posted 28 December 2011 - 20:15

thanks, i have it committed



timer list not very neat.


I have implemented a fix for this, but I don't know how to create the git patch from my repository with TortoiseGIT.
Could anybody tell me how to do that?


OK, Google (possibly) helped...
Here is the patch for the scrambled timer list: if something is wrong in the packaging, please let me know.



Re: OpenWebif [E2OpenPlugins] #428 Rincewind

  • Member
  • 25 posts

+12
Neutral

Posted 28 December 2011 - 20:31

Now that I learned, I start enjoying it ;-)

Here's another patch that fixes the pulldown directories menu in the movies list, as mentioned in a previous message.

Attached Files



Re: OpenWebif [E2OpenPlugins] #429 nobody9

  • Senior Member
  • 187 posts

+76
Good

Posted 28 December 2011 - 20:40

thanks, also committed,
enough for today for me.


Now that I learned, I start enjoying it ;-)

Here's another patch that fixes the pulldown directories menu in the movies list, as mentioned in a previous message.



Re: OpenWebif [E2OpenPlugins] #430 Hump

  • Senior Member
  • 88 posts

+3
Neutral

Posted 28 December 2011 - 20:41

There is still a bug in "stream>movies".

Only the main directory "hdd/movie" can be accessed. When selecting a subdirectories, following error occurs: " Error 404: Page not found - The requested URL was not found on this server.

Thanks. :)

Edit: You guys are fast. While I was writing this post, the fix was committet! - Thanks again!

Edited by Hump, 28 December 2011 - 20:43.


Re: OpenWebif [E2OpenPlugins] #431 dylantje

  • Senior Member
  • 2,406 posts

+19
Neutral

Posted 28 December 2011 - 23:26

He hello all,

Go on with the great work.
When it is in the [ test ]feed?
So we can try it?

Mut@nt HD51 with PLi 7.2 release - Cccam, PLi HD skin, HDD-1,5TB, USB DVB-T,4GB usb backupsuite. AB: harmony smartcontrol 

XTrend ET6500 with PLi 7.2 release - PLi HD skin, Stream remote, 2GB usb backupsuite. AB: Harmony 650 , 1x zgemma h2h with Cccam.

Met een USB Dvb-T Triax 78cm, 2x ALPS quatro LNB /19,2 en 23,5, SAB GSO-multiswitch: 4 sat. in 12 out. 

 


Re: OpenWebif [E2OpenPlugins] #432 bacicciosat

  • Senior Member
  • 540 posts

+100
Excellent

Posted 28 December 2011 - 23:27

now openwebif starts on 'Current' tab which is not very useful imho (you can't even start streaming of a current channel from it...)
starting on a 'Bouquets' tab I find far much convenient - I (like many /?/ others) use webif for streaming purposes most times


Finally i realized that this is a good idea. :D
done !

Re: OpenWebif [E2OpenPlugins] #433 tilleke

  • Senior Member
  • 50 posts

+5
Neutral

Posted 29 December 2011 - 02:53

@nobody9

sorry to bother again but there is still something which is not compatible with the original WebIf what regards stream.m3u.

I didn't notice this difference before because I was convinced the difference was only the #EXTINF-line. However, after testing with your latest commits, I noted a difference what regards the Service Reference.

This is the output using wget with original WebIf (I am using port 8888) and OpenWebIf (port 8088):

OpenDreambox 1.6.0 dm500hd

ORIGINAL WEBIF
dm500hd login: root
root@dm500hd:~# wget -q -O - http://127.0.0.1:8888/web/stream.m3u
#EXTM3U
#EXTVLCOPT--http-reconnect=true
http://127.0.0.1:8001/


OPENWEBIF
root@dm500hd:~# wget -q -O - http://127.0.0.1:8088/web/stream.m3u
#EXTM3U
#EXTVLCOPT--http-reconnect=true
http://127.0.0.1:800...F:820000:0:0:0:
root@dm500hd:~#


as you can see, using original WebIf the service-reference is not included while with OpenWebIf the service-reference is added.

I don't know how this web-stuff works but with original web-if, if I save the stream.m3u and then open it in a text-editor, then the service-reference has been added (later in the process somehow, perhaps during download, but in the beginning the service-reference is not included as you can see from the wget-command). This is the reason why it is possible to stream directly from the browser opening the file with VLC.

In OpenWebIf, the service-reference is ALREADY included from the beginning and is not added later in the process. In any case, streaming will work by opening the resulting m3u-file with VLC since service-reference is present.

To make OpenWebIf compatible (and not to break my code), you would need to change your code in stream.py (I think - I don't know python) as follows:

from:
response = "#EXTM3U\n#EXTVLCOPT--http-reconnect=true\nhttp://%s:8001/%s\n" % (request.getRequestHostname(), sRef)

to:
response = "#EXTM3U\n#EXTVLCOPT--http-reconnect=true\nhttp://%s:8001/\n" % (request.getRequestHostname())

BUT then in OpenWebIf streaming will not work by opening stream.m3u-file with VLC since service-reference is not included unless you find out how it is done in original WebIf.

If it is difficult, I can change my code in my old apps (new apps = no problem) to make it easier for you. What regards other third-party developers, I cannot speak for them but I believe most developers are simply using "web/streamcurrent.m3u" for launching streaming. If you can fix it so it works like in original WebIF, that would be nice otherwise nevermind.

rgds,
tilleke

Edited by tilleke, 29 December 2011 - 02:54.


Re: OpenWebif [E2OpenPlugins] #434 nobody9

  • Senior Member
  • 187 posts

+76
Good

Posted 29 December 2011 - 04:41

@tilleke,
i can´t change this.
The external streamproxy sends you without a ServiceRef the aktive channel,
the internal Streamserver from Pli sends you nothing.
So at the moment i must send the Serviceref in the request.

@Pli Team,
can you change the internal streaming server that they stream the active channel when no ServiceRef is given


@nobody9

sorry to bother again but there is still something which is not compatible with the original WebIf what regards stream.m3u.

I didn't notice this difference before because I was convinced the difference was only the #EXTINF-line. However, after testing with your latest commits, I noted a difference what regards the Service Reference.

This is the output using wget with original WebIf (I am using port 8888) and OpenWebIf (port 8088):

OpenDreambox 1.6.0 dm500hd

ORIGINAL WEBIF
dm500hd login: root
root@dm500hd:~# wget -q -O - http://127.0.0.1:8888/web/stream.m3u
#EXTM3U
#EXTVLCOPT--http-reconnect=true
http://127.0.0.1:8001/


OPENWEBIF
root@dm500hd:~# wget -q -O - http://127.0.0.1:8088/web/stream.m3u
#EXTM3U
#EXTVLCOPT--http-reconnect=true
http://127.0.0.1:800...F:820000:0:0:0:
root@dm500hd:~#


as you can see, using original WebIf the service-reference is not included while with OpenWebIf the service-reference is added.

I don't know how this web-stuff works but with original web-if, if I save the stream.m3u and then open it in a text-editor, then the service-reference has been added (later in the process somehow, perhaps during download, but in the beginning the service-reference is not included as you can see from the wget-command). This is the reason why it is possible to stream directly from the browser opening the file with VLC.

In OpenWebIf, the service-reference is ALREADY included from the beginning and is not added later in the process. In any case, streaming will work by opening the resulting m3u-file with VLC since service-reference is present.

To make OpenWebIf compatible (and not to break my code), you would need to change your code in stream.py (I think - I don't know python) as follows:

from:
response = "#EXTM3U\n#EXTVLCOPT--http-reconnect=true\nhttp://%s:8001/%s\n" % (request.getRequestHostname(), sRef)

to:
response = "#EXTM3U\n#EXTVLCOPT--http-reconnect=true\nhttp://%s:8001/\n" % (request.getRequestHostname())

BUT then in OpenWebIf streaming will not work by opening stream.m3u-file with VLC since service-reference is not included unless you find out how it is done in original WebIf.

If it is difficult, I can change my code in my old apps (new apps = no problem) to make it easier for you. What regards other third-party developers, I cannot speak for them but I believe most developers are simply using "web/streamcurrent.m3u" for launching streaming. If you can fix it so it works like in original WebIF, that would be nice otherwise nevermind.

rgds,
tilleke



Re: OpenWebif [E2OpenPlugins] #435 nobody9

  • Senior Member
  • 187 posts

+76
Good

Posted 29 December 2011 - 04:53

i can´t change my old topic,

The documentation
for /web/stream.m3u is always with Service Ref.
normally you must use without /web/streamcurrent.m3u.

So it´s not a issue from OpenWebif,

it´s a issue from Streamproxy/Internal Streamserver and the old Webif.

Re: OpenWebif [E2OpenPlugins] #436 nobody9

  • Senior Member
  • 187 posts

+76
Good

Posted 29 December 2011 - 06:33

Stop,
i make a mistake,
i changed the Code, only when requested /web/streamcurrent.m3u the ServiceRef will append.
after this change your app works again.



@tilleke,
you add the serviceref in your program.
So when the webif send you the request without it, all works fine.


@Pli team

you must change nothing


@tilleke,
i can´t change this.
The external streamproxy sends you without a ServiceRef the aktive channel,
the internal Streamserver from Pli sends you nothing.
So at the moment i must send the Serviceref in the request.

@Pli Team,
can you change the internal streaming server that they stream the active channel when no ServiceRef is given



Re: OpenWebif [E2OpenPlugins] #437 bacicciosat

  • Senior Member
  • 540 posts

+100
Excellent

Posted 29 December 2011 - 09:20

@Pli team
you must change nothing



But, do you have understood that OpenWebif is not a Pli application ?
Pli code don't care here. Streaming have to work on all the images.

Re: OpenWebif [E2OpenPlugins] #438 nobody9

  • Senior Member
  • 187 posts

+76
Good

Posted 29 December 2011 - 09:51

I understood this, OpenWebif is not a Pli application.
OpenWebif is from all of us to all of us.

The biggest problem with the streaming - functions is , the big dependent on the e2-api.



@Pli team
you must change nothing



But, do you have understood that OpenWebif is not a Pli application ?
Pli code don't care here. Streaming have to work on all the images.



Re: OpenWebif [E2OpenPlugins] #439 tilleke

  • Senior Member
  • 50 posts

+5
Neutral

Posted 29 December 2011 - 11:21

@nobody9

Thanks for resolving streaming-issue. Tested with latest commit and the streaming part (stream.m3u) seems to be identical as to original WebIf.

Good job - in this way compatibility should be 100%.

rgds,
tilleke

Re: OpenWebif [E2OpenPlugins] #440 blzr

  • PLi® Core member
  • 2,270 posts

+118
Excellent

Posted 29 December 2011 - 11:29


now openwebif starts on 'Current' tab which is not very useful imho (you can't even start streaming of a current channel from it...)
starting on a 'Bouquets' tab I find far much convenient - I (like many /?/ others) use webif for streaming purposes most times


Finally i realized that this is a good idea. :D
done !

nice, thanks...
so maybe I'll try to convince you to remove (i)nfo icon too? ;)
I know nothing about web designing, so I'm just throwing some general ideas,
so, attached three jpegs:
1. op_wbif_now channels list as it looks now
and, for comparsion (I assume you'll add link to description under event title/?/):
2. op_wbif_mod1 - one (i) icon left (more as an indicator that there is description availlable for given service)
3. op_wbif_mod2 - with (i) icon removed completely (I like it best, the more minimalistic look the better ;))
in both my drafts I increased spacing between each service and deleted theese various white lines
It would look much more clearer (and nicer :P) imho
//but of course I know that easier said than done...
and what do you think?

Attached Files


True sarcasm doesn't need green font...


15 user(s) are reading this topic

0 members, 15 guests, 0 anonymous users