Bug 1546583 - Plugin doesn't pass "user" and "remotename" options to pppd
Summary: Plugin doesn't pass "user" and "remotename" options to pppd
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: NetworkManager-pptp
Version: 27
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Lubomir Rintel
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-02-19 00:08 UTC by Alexander
Modified: 2018-04-04 15:52 UTC (History)
5 users (show)

Fixed In Version: NetworkManager-pptp-1.2.6-1.fc28
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-04-04 15:52:19 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch for passing "user" and "remotename" options to pppd (1.00 KB, patch)
2018-02-19 00:08 UTC, Alexander
no flags Details | Diff
Patch against master in "git format-path" format (1.21 KB, patch)
2018-03-26 12:44 UTC, Alexander
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
GNOME Bugzilla 794695 0 None None None 2018-03-29 05:39:54 UTC

Description Alexander 2018-02-19 00:08:51 UTC
Created attachment 1397728 [details]
Patch for passing "user" and "remotename" options to pppd

Description of problem:
Plugin doesn't pass "user" and "remotename" options to pppd

Version-Release number of selected component (if applicable):
Plugin doesn't pass "user" and "remotename" options to pppd

How reproducible:
Normal use

Steps to Reproduce:
1. Configure VPN connection through NetworkManager pptp plugin
2. Try to establish connection

Actual results:
pppd can't establish connection

Expected results:
Connection must be established

Additional info:
Need to add attached patch and modify spec as
--- NetworkManager-pptp.spec_orig	2018-02-19 03:01:12.762515208 +0300
+++ NetworkManager-pptp.spec	2018-02-12 03:04:22.564243321 +0300
@@ -8,6 +8,8 @@
 Group:     System Environment/Base
 Source0:   https://download.gnome.org/sources/NetworkManager-pptp/1.2/%{name}-%{version}.tar.xz
 
+Patch0001: user_remotename.patch
+
 %global ppp_version %(sed -n 's/^#define\\s*VERSION\\s*"\\([^\\s]*\\)"$/\\1/p' %{_includedir}/pppd/patchlevel.h 2>/dev/null | grep . || echo bad)
 
 BuildRequires: gtk3-devel
@@ -47,6 +49,7 @@
 
 %prep
 %setup -q
+%autopatch -p2
 
 %build
 %configure \

Comment 1 Thomas Haller 2018-02-19 10:25:53 UTC
Please create patches against upstream first (https://git.gnome.org/browse/network-manager-pptp , against branch "master".

Please create patches with `git format-patch`, and provide a good commit message (https://chris.beams.io/posts/git-commit/#seven-rules might be a good help)

It's fine to attach the patch on the this downstream bug-tracker.

If you have difficulties using git, please state your preferred "Firstname Surname <email>" for the authorship of the patch here and somebody will create the upstream commit.

Thanks.

Comment 2 Alexander 2018-03-26 12:44:30 UTC
Created attachment 1413102 [details]
Patch against master in "git format-path" format

[PATCH] Fix passing "user" field and add passing "remotename"

Comment 3 Thomas Haller 2018-03-26 13:03:47 UTC
Comment on attachment 1413102 [details]
Patch against master in "git format-path" format

Same patch is upstream at https://bugzilla.gnome.org/show_bug.cgi?id=794695

Comment 5 Fedora Update System 2018-03-29 08:53:33 UTC
NetworkManager-pptp-1.2.6-1.fc28 has been submitted as an update to Fedora 28. https://bodhi.fedoraproject.org/updates/FEDORA-2018-3ddd53ddbc

Comment 6 Fedora Update System 2018-03-29 13:58:38 UTC
NetworkManager-pptp-1.2.6-1.fc28 has been pushed to the Fedora 28 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2018-3ddd53ddbc

Comment 7 Fedora Update System 2018-04-04 15:52:19 UTC
NetworkManager-pptp-1.2.6-1.fc28 has been pushed to the Fedora 28 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.