Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Platinsport site #4016

Open
wants to merge 4 commits into
base: Beta
Choose a base branch
from
Open

Platinsport site #4016

wants to merge 4 commits into from

Conversation

caveman77
Copy link

Hello
please find here a new site : platinsport.

Any question, just let me know.

A bientot.
Christian.

@sizanic
Copy link
Collaborator

sizanic commented Feb 25, 2023

Salut,

Et merci pour cette source 👍🏻

On va vérifier l'interet d'ajouter une dépendance avec Plexus.
-> As-tu l'adresse d'un repo officiel de Plexus ?

Et le zip que tu as ajouté dans ta PR est de trop 🤔

@sizanic
Copy link
Collaborator

sizanic commented Feb 25, 2023

Le sujet Plexus a déjà été abordé.

Il faut voir si nous allons revoir notre position :
#2212 (comment)

@Arias800 , tu pourrais nous en dire un peu plus ?

@caveman77
Copy link
Author

Merci @sizanic pour le retour. Effectivement le zip peut etre supprimé.
Plexus a le gros avantage de fournir les interfaces sopcast et acestream. C'est certainement en perte de vitesse. Néanmoins je trouve la qualité video bien au dessus avec les flux acestream.

Coté répo j'ai toujours installé Plexus depuis le repo tvaddons.
D'ailleurs ils semblent avoir repris le source a leur compte. Le git initial est probablement celui la: https://github.com/enen92/program.plexus, le git tvaddons ici: https://github.com/staycanuca/program.plexus

A bientot!

@TmpName
Copy link
Collaborator

TmpName commented Feb 26, 2023

Ca fait un moment que les fichiers ont pas été mit a jour ^^, en tout cas si ca fonctionne encore c'est stable.

Par contre je mettrais ca en optionnel.

@caveman77
Copy link
Author

ca fait longtemps que j utilise cet addon (depuis 2016). Surtout avec des liens acestream. Il marche bien et permet de nombreuses configurations (ace server local / distant par ex). Le server ace qui est embarqué dans l'instalation plugin est un peu vieux. Il vaut mieux en telecharger un plus recent.

@sizanic
Copy link
Collaborator

sizanic commented Feb 26, 2023

Le souci n'est pas de savoir si il fonctionne bien ou pas.
Mais le souci est qu'il ne s'agit pas ici de streaming, mais de p2p.
Et dans le passé nous avions déjà refusé de proposer des liens de ce type à cause de cette raison.

Il faut donc voir si nous revoyons notre position par rapport à ça.

@caveman77
Copy link
Author

Bien noté.

@sizanic
Copy link
Collaborator

sizanic commented Feb 26, 2023

c'est de la technologie p2p mais vers des serveurs, pas vers des particuliers ?
Et dans un seul sens je présume, vu qu'on ne va rien envoyer en retour.

@caveman77
Copy link
Author

le server ace qui est piloté par plexus va télécharger du contenu mais aussi en partager.

@TmpName
Copy link
Collaborator

TmpName commented Feb 27, 2023

@sizanic ha oui mince, tu as raison, j'avais zappé.
Ben le soucis, c'est que Vstream est un addon de streaming, pas de p2p, c'est le mode de fonctionnement que tout le monde connais, je me vois pas dire du jour au lendemain que Vstream fonctionne a 80% en streaming et 20% en p2p.

Et en plus on a aucun controle sur le "serveur", il faut aussi expliquer aux utilisateurs qu'il doivent l'installer, et le regler selon leurs "normes de securitees". Ca fait beacoup par rapport a la simplicité de fonctionnement de Vstream (ca cherche un lien via un moteur WEB, et l'affiche via protocole HTTP)

@sizanic
Copy link
Collaborator

sizanic commented Mar 3, 2023

aucun controle sur le "serveur", il faut aussi expliquer aux utilisateurs qu'il doivent l'installer

Je n'avais même pas bien compris qu'il fallait installer un serveur.
Et qu'il y a donc forcement des échanges p2p dans les 2 sens.

Alors dans ce cas, c'est plutôt rédhibitoire.

@TmpName
Copy link
Collaborator

TmpName commented Mar 3, 2023

Alors dans cas, c'est plutôt rédhibitoire.

Pareil, moi ca me gene de mettre ca, car ca va demander a mettre en place des explications.
D'un autre coté, il a fait tout le boulot, c'est con de jeter ? et avec un moyen de cacher la source, sauf si l'utilisateur "sait ce qu'il fait" qu'il se debrouille avec, et l'active lui meme ?

Le site est vraiment fiable et stable ? Car en general ce genre de site demande de modifier le code tout les mois

@caveman77
Copy link
Author

Le site est stable depuis longtemps. D'ailleurs j en suis le premier étonné.

Une option serait peut etre de désactiver le site par défaut et de laisser l'utilisateur l'activer dans le menu setting de l addon.

@TmpName
Copy link
Collaborator

TmpName commented Mar 6, 2023

Alors je t'avoue que la ca pose probleme ^^.
Depuis le debut, Vstream c'est tout sauf le p2p, si on commence a ouvrir la porte ca va etre le bordel.

Du coup, ca nous gene pas mal de l'inclure de maniere officielle dans Vstream, et si on fait une option pour l'activer, personne ne s'en servira.

Reste la possiblité de proposer la source, mais non officiellement, via un topic special par exemple, du coup les gens devront l'installer eux meme, ca va faire du tri naturellement ?




# try:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi gardé du code commenté ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

effectivement. aucun interet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants