<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>27665</bug_id>
          
          <creation_ts>2012-08-24 19:25:35 +0400</creation_ts>
          <short_desc>NetworkManager doesn&apos;t honor specified methods for the PPTP VPN links</short_desc>
          <delta_ts>2013-02-21 18:16:14 +0400</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>NetworkManager-pptp</component>
          <version>unstable</version>
          <rep_platform>all</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="(GalaxyMaster)">galaxy4public</reporter>
          <assigned_to name="Mikhail Efremov">sem</assigned_to>
          <cc>sem</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>132972</commentid>
    <comment_count>0</comment_count>
    <who name="(GalaxyMaster)">galaxy4public</who>
    <bug_when>2012-08-24 19:25:35 +0400</bug_when>
    <thetext>The ipv4.ignore-auto-dns setting is ignored for PPTP connections, hence it&apos;s impossible to setup a PPTP connection that doesn&apos;t use the server supplied DNS servers.

The reason for this is that NetworkManager-pptp is written is such a way that it uses hardcoded options for pppd, e.g. &apos;usepeerdns&apos;, and when pppd establishes the connection the /etc/ppp/ip-up script is executed.  The script will overwrite /etc/resolv.conf if &apos;usepeerdns&apos; was supplied to pppd.

I&apos;m attaching a quick fix that makes the ipv4.ignore-auto-dns setting to be honored by NetworkManager-pptp.  The attached patch is a bit dirty, but it works.  Feel free to add error checking and stuff, or you can leave it as is since it&apos;s a not that critical part.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132973</commentid>
    <comment_count>1</comment_count>
      <attachid>5548</attachid>
    <who name="(GalaxyMaster)">galaxy4public</who>
    <bug_when>2012-08-24 19:28:08 +0400</bug_when>
    <thetext>Created attachment 5548
conditionaly enable usepeerdns to pppd subprocess</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132974</commentid>
    <comment_count>2</comment_count>
    <who name="(GalaxyMaster)">galaxy4public</who>
    <bug_when>2012-08-24 19:40:12 +0400</bug_when>
    <thetext>I&apos;ve also contacted Dan Williams at RH (one of the current upstream maintainers) and provided him with the patch file, JFYI.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132976</commentid>
    <comment_count>3</comment_count>
    <who name="Mikhail Efremov">sem</who>
    <bug_when>2012-08-24 20:43:40 +0400</bug_when>
    <thetext>Thanks, this patch looks reasonable. Maybe I&apos;ll do some cleanup for it and then apply it.
Or maybe Dan commit the patch and then I&apos;ll just take it from upstream&apos;s git ;)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>138016</commentid>
    <comment_count>4</comment_count>
    <who name="Mikhail Efremov">sem</who>
    <bug_when>2013-02-19 16:13:06 +0400</bug_when>
    <thetext>Is there any news from Dan? I don&apos;t see such commit in the upstream git.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>138082</commentid>
    <comment_count>5</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2013-02-21 18:16:14 +0400</bug_when>
    <thetext>NetworkManager-pptp-0.9.8.0-alt1 -&gt; sisyphus:

* Thu Feb 21 2013 Mikhail Efremov &lt;sem@altlinux&gt; 0.9.8.0-alt1
- Treat warrnings as errors again.
- Conditionally enable &apos;usepeerdns&apos; pppd option (by GalaxyMaster)
    (closes: #27665).
- Updated to 0.9.8.0.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>5548</attachid>
            <date>2012-08-24 19:28:08 +0400</date>
            <delta_ts>2012-08-24 19:28:08 +0400</delta_ts>
            <desc>conditionaly enable usepeerdns to pppd subprocess</desc>
            <filename>NetworkManager-pptp-0.9.6.0-usepeerdns.patch</filename>
            <type>text/plain</type>
            <size>1080</size>
            <attacher name="(GalaxyMaster)">galaxy4public</attacher>
            
              <data encoding="base64">LS0tIE5ldHdvcmtNYW5hZ2VyLXBwdHAtMC45LjYuMC5vcmlnL3NyYy9ubS1wcHRwLXNlcnZpY2Uu
YwkyMDEyLTA4LTA3IDE2OjU5OjM2LjAwMDAwMDAwMCArMDAwMAorKysgTmV0d29ya01hbmFnZXIt
cHB0cC0wLjkuNi4wL3NyYy9ubS1wcHRwLXNlcnZpY2UuYwkyMDEyLTA4LTI0IDE1OjA0OjU1LjA2
ODk4MjQ3NSArMDAwMApAQCAtODQ4LDcgKzg0OCwxNSBAQCBjb25zdHJ1Y3RfcHBwZF9hcmdzIChO
TVBwdHBQbHVnaW4gKnBsdWdpCiAKIAlnX3B0cl9hcnJheV9hZGQgKGFyZ3MsIChncG9pbnRlcikg
Z19zdHJkdXAgKCJub2RldGFjaCIpKTsKIAlnX3B0cl9hcnJheV9hZGQgKGFyZ3MsIChncG9pbnRl
cikgZ19zdHJkdXAgKCJsb2NrIikpOwotCWdfcHRyX2FycmF5X2FkZCAoYXJncywgKGdwb2ludGVy
KSBnX3N0cmR1cCAoInVzZXBlZXJkbnMiKSk7CisKKwkvKiBYWFg6IChHTSk6IHRoaXMgaXMgYSBo
YWNrIHRvIGVuc3VyZSB0aGF0IHBwcGQgaXNuJ3QgcHJvdmlkZWQKKwkJd2l0aCB1c2VwZWVyZG5z
IHdoZW4gTk0ncyBjb25maWdzIHNheSB0byBmb3JnZXQgYWNxdWlyZWQKKwkJTlNlcy4gIFdpdGgg
dXNlcGVlcmRucyBhbmQgdGhlIGN1cnJlbnQgaW1wbGVtZW50YXRpb24gb2YKKwkJdGhpcyBwbHVn
aW4sIHBwcGQgd2lsbCBleGVjdXRlIC9ldGMvcHBwL2lwLXVwIGFuZCB3aWxsCisJCXJlcGxhY2Ug
L2V0Yy9yZXNvbHYuY29uZi4gKi8KKwlpZiAoIW5tX3NldHRpbmdfaXA0X2NvbmZpZ19nZXRfaWdu
b3JlX2F1dG9fZG5zICgoTk1TZXR0aW5nSVA0Q29uZmlnICopIG5tX2Nvbm5lY3Rpb25fZ2V0X3Nl
dHRpbmcgKE5NX1BQVFBfUExVR0lOX0dFVF9QUklWQVRFIChwbHVnaW4pLT5jb25uZWN0aW9uLCBO
TV9UWVBFX1NFVFRJTkdfSVA0X0NPTkZJRykpKQorCQlnX3B0cl9hcnJheV9hZGQgKGFyZ3MsIChn
cG9pbnRlcikgZ19zdHJkdXAgKCJ1c2VwZWVyZG5zIikpOworCiAJZ19wdHJfYXJyYXlfYWRkIChh
cmdzLCAoZ3BvaW50ZXIpIGdfc3RyZHVwICgibm9pcGRlZmF1bHQiKSk7CiAJZ19wdHJfYXJyYXlf
YWRkIChhcmdzLCAoZ3BvaW50ZXIpIGdfc3RyZHVwICgibm9kZWZhdWx0cm91dGUiKSk7CiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>