On Wed, 19 Feb 2020 at 12:12:51 +0100, Solene Rapenne wrote:
> On Wed, Feb 19, 2020 at 11:35:52AM +0100, Solene Rapenne wrote:
> > Hi,
> > 
> > smtube uses the website tonvid.com but it uses http:// by default, this 
> > patch
> > will make it use https://tonvid.com instead.
> > 
> > (and a crash report below the patch)
> > 
> > 
> 
> better patch doing https instead of wwws
> 
> many thanks bin on irc who spotted it
> 
> Index: Makefile
> ===================================================================
> RCS file: /data/cvs/ports/x11/smtube/Makefile,v
> retrieving revision 1.35
> diff -u -p -r1.35 Makefile
> --- Makefile  15 Jul 2019 13:21:51 -0000      1.35
> +++ Makefile  19 Feb 2020 10:30:13 -0000
> @@ -5,6 +5,7 @@ USE_WXNEEDED =                Yes
>  COMMENT =            browse and download YouTube videos
>  
>  DISTNAME =           smtube-19.6.0
> +REVISION =           0
>  
>  CATEGORIES =         multimedia x11
>  
> Index: patches/patch-src_browserwindow_cpp
> ===================================================================
> RCS file: patches/patch-src_browserwindow_cpp
> diff -N patches/patch-src_browserwindow_cpp
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-src_browserwindow_cpp       19 Feb 2020 11:12:01 -0000
> @@ -0,0 +1,23 @@
> +$OpenBSD$
> +
> +Index: src/browserwindow.cpp
> +--- src/browserwindow.cpp.orig
> ++++ src/browserwindow.cpp
> +@@ -70,7 +70,7 @@ BrowserWindow::BrowserWindow(const QString & config_pa
> + {
> +     setWindowTitle("SMTube");
> +     setWindowIcon(QPixmap(":/icons/smtube.png"));
> +-    home_page = "http://www.tonvid.com/";;
> ++    home_page = "https://www.tonvid.com/";;
> + 
> +     settings = new QSettings(config_path + "/smtube2.ini", 
> QSettings::IniFormat, this);
> + 
> +@@ -442,7 +442,7 @@ void BrowserWindow::openYTUrl(QString title, QString e
> +     if (current_player == WebBrowser) {
> +             QString u = url;
> +             #if 1
> +-            u = "http://www.tonvid.com/video.php?u="; + 
> url.toUtf8().toBase64().toPercentEncoding() +
> ++            u = "https://www.tonvid.com/video.php?u="; + 
> url.toUtf8().toBase64().toPercentEncoding() +
> +             "&t=" + title.toUtf8().toBase64().toPercentEncoding();
> +             #endif
> +             QDesktopServices::openUrl(u);
> 

There is a new version of smtube, I will check this change and send a diff with
that included if is not already in.

Thanks.

-- 

                - gonzalo

Reply via email to