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