[Touch-packages] [Bug 1825021] Re: apt's dpkgpm.cc WriteApportReport function should gather more data

2020-10-14 Thread Matthieu Clemenceau
** Tags added: fr-238

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apt in Ubuntu.
https://bugs.launchpad.net/bugs/1825021

Title:
  apt's dpkgpm.cc WriteApportReport function should gather more data

Status in apt package in Ubuntu:
  New
Status in apt source package in Eoan:
  Won't Fix

Bug description:
  While fixing bug 1813464 it occurred to me that it is not obvious that
  the dist-upgrader tarball includes its own copy of invoke-rc.d and
  that this has been complicated to debug more than once. The apport
  information in that report was gathered by apt's dpkgpm.cc code and I
  think we should extend that to gather additional information from the
  environment at the time that the package installation failed. For
  example the dist-upgrader sets the following:

  os.environ["PATH"] = "%s:%s" % (os.getcwd()+"/imported",
  os.environ["PATH"])

  Gathering path from the environment would have made diagnosing bug
  1813464 a lot easier. I tested modifying dpkgpm.cc with the following:

 if(const char* env_p = std::getenv("PATH"))
fprintf(report, "EnvPATH: %s\n", env_p);

  I then installed the crash-in-postinst package from the daisy-plucker-
  seeds PPA and received a crash report with the EnvPATH key and value.
  Its worth noting that when apport gathers environment information only
  a white listed set of keys is gathered to prevent disclosing
  potentially sensitive information. So we should look at what
  environment variables the dist-upgrader sets and gather those at a
  minimum.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1825021/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp


[Touch-packages] [Bug 1825021] Re: apt's dpkgpm.cc WriteApportReport function should gather more data

2020-08-18 Thread Brian Murray
eoan has reached end of life, so this bug will not be fixed for that
release

** Changed in: apt (Ubuntu Eoan)
   Status: New => Won't Fix

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apt in Ubuntu.
https://bugs.launchpad.net/bugs/1825021

Title:
  apt's dpkgpm.cc WriteApportReport function should gather more data

Status in apt package in Ubuntu:
  New
Status in apt source package in Eoan:
  Won't Fix

Bug description:
  While fixing bug 1813464 it occurred to me that it is not obvious that
  the dist-upgrader tarball includes its own copy of invoke-rc.d and
  that this has been complicated to debug more than once. The apport
  information in that report was gathered by apt's dpkgpm.cc code and I
  think we should extend that to gather additional information from the
  environment at the time that the package installation failed. For
  example the dist-upgrader sets the following:

  os.environ["PATH"] = "%s:%s" % (os.getcwd()+"/imported",
  os.environ["PATH"])

  Gathering path from the environment would have made diagnosing bug
  1813464 a lot easier. I tested modifying dpkgpm.cc with the following:

 if(const char* env_p = std::getenv("PATH"))
fprintf(report, "EnvPATH: %s\n", env_p);

  I then installed the crash-in-postinst package from the daisy-plucker-
  seeds PPA and received a crash report with the EnvPATH key and value.
  Its worth noting that when apport gathers environment information only
  a white listed set of keys is gathered to prevent disclosing
  potentially sensitive information. So we should look at what
  environment variables the dist-upgrader sets and gather those at a
  minimum.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1825021/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp


[Touch-packages] [Bug 1825021] Re: apt's dpkgpm.cc WriteApportReport function should gather more data

2019-05-09 Thread Steve Langasek
** Also affects: apt (Ubuntu Eoan)
   Importance: High
   Status: New

** Tags removed: rls-ee-incoming

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apt in Ubuntu.
https://bugs.launchpad.net/bugs/1825021

Title:
  apt's dpkgpm.cc WriteApportReport function should gather more data

Status in apt package in Ubuntu:
  New
Status in apt source package in Eoan:
  New

Bug description:
  While fixing bug 1813464 it occurred to me that it is not obvious that
  the dist-upgrader tarball includes its own copy of invoke-rc.d and
  that this has been complicated to debug more than once. The apport
  information in that report was gathered by apt's dpkgpm.cc code and I
  think we should extend that to gather additional information from the
  environment at the time that the package installation failed. For
  example the dist-upgrader sets the following:

  os.environ["PATH"] = "%s:%s" % (os.getcwd()+"/imported",
  os.environ["PATH"])

  Gathering path from the environment would have made diagnosing bug
  1813464 a lot easier. I tested modifying dpkgpm.cc with the following:

 if(const char* env_p = std::getenv("PATH"))
fprintf(report, "EnvPATH: %s\n", env_p);

  I then installed the crash-in-postinst package from the daisy-plucker-
  seeds PPA and received a crash report with the EnvPATH key and value.
  Its worth noting that when apport gathers environment information only
  a white listed set of keys is gathered to prevent disclosing
  potentially sensitive information. So we should look at what
  environment variables the dist-upgrader sets and gather those at a
  minimum.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1825021/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp


[Touch-packages] [Bug 1825021] Re: apt's dpkgpm.cc WriteApportReport function should gather more data

2019-04-19 Thread David Kalnischkies
No idea about Ubuntu specifically, but it should be in upstream since apt 
1.8.0~alpha3 release on Tue, 18 Dec 2018 15:02:11 +0100.
See also: https://salsa.debian.org/apt-team/apt/merge_requests/38

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apt in Ubuntu.
https://bugs.launchpad.net/bugs/1825021

Title:
  apt's dpkgpm.cc WriteApportReport function should gather more data

Status in apt package in Ubuntu:
  New

Bug description:
  While fixing bug 1813464 it occurred to me that it is not obvious that
  the dist-upgrader tarball includes its own copy of invoke-rc.d and
  that this has been complicated to debug more than once. The apport
  information in that report was gathered by apt's dpkgpm.cc code and I
  think we should extend that to gather additional information from the
  environment at the time that the package installation failed. For
  example the dist-upgrader sets the following:

  os.environ["PATH"] = "%s:%s" % (os.getcwd()+"/imported",
  os.environ["PATH"])

  Gathering path from the environment would have made diagnosing bug
  1813464 a lot easier. I tested modifying dpkgpm.cc with the following:

 if(const char* env_p = std::getenv("PATH"))
fprintf(report, "EnvPATH: %s\n", env_p);

  I then installed the crash-in-postinst package from the daisy-plucker-
  seeds PPA and received a crash report with the EnvPATH key and value.
  Its worth noting that when apport gathers environment information only
  a white listed set of keys is gathered to prevent disclosing
  potentially sensitive information. So we should look at what
  environment variables the dist-upgrader sets and gather those at a
  minimum.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1825021/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp


Re: [Touch-packages] [Bug 1825021] Re: apt's dpkgpm.cc WriteApportReport function should gather more data

2019-04-19 Thread Brian Murray
On Tue, Apr 16, 2019 at 04:36:04PM -, Julian Andres Klode wrote:
> FWIW, apt actually overrides PATH since bionic or cosmic or something
> like that.

If that is the case then how could have the original bug been an issue?

--
Brian Murray

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apt in Ubuntu.
https://bugs.launchpad.net/bugs/1825021

Title:
  apt's dpkgpm.cc WriteApportReport function should gather more data

Status in apt package in Ubuntu:
  New

Bug description:
  While fixing bug 1813464 it occurred to me that it is not obvious that
  the dist-upgrader tarball includes its own copy of invoke-rc.d and
  that this has been complicated to debug more than once. The apport
  information in that report was gathered by apt's dpkgpm.cc code and I
  think we should extend that to gather additional information from the
  environment at the time that the package installation failed. For
  example the dist-upgrader sets the following:

  os.environ["PATH"] = "%s:%s" % (os.getcwd()+"/imported",
  os.environ["PATH"])

  Gathering path from the environment would have made diagnosing bug
  1813464 a lot easier. I tested modifying dpkgpm.cc with the following:

 if(const char* env_p = std::getenv("PATH"))
fprintf(report, "EnvPATH: %s\n", env_p);

  I then installed the crash-in-postinst package from the daisy-plucker-
  seeds PPA and received a crash report with the EnvPATH key and value.
  Its worth noting that when apport gathers environment information only
  a white listed set of keys is gathered to prevent disclosing
  potentially sensitive information. So we should look at what
  environment variables the dist-upgrader sets and gather those at a
  minimum.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1825021/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp


[Touch-packages] [Bug 1825021] Re: apt's dpkgpm.cc WriteApportReport function should gather more data

2019-04-17 Thread Francis Ginther
** Tags added: id-5cb602c6889ce851761205ca

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apt in Ubuntu.
https://bugs.launchpad.net/bugs/1825021

Title:
  apt's dpkgpm.cc WriteApportReport function should gather more data

Status in apt package in Ubuntu:
  New

Bug description:
  While fixing bug 1813464 it occurred to me that it is not obvious that
  the dist-upgrader tarball includes its own copy of invoke-rc.d and
  that this has been complicated to debug more than once. The apport
  information in that report was gathered by apt's dpkgpm.cc code and I
  think we should extend that to gather additional information from the
  environment at the time that the package installation failed. For
  example the dist-upgrader sets the following:

  os.environ["PATH"] = "%s:%s" % (os.getcwd()+"/imported",
  os.environ["PATH"])

  Gathering path from the environment would have made diagnosing bug
  1813464 a lot easier. I tested modifying dpkgpm.cc with the following:

 if(const char* env_p = std::getenv("PATH"))
fprintf(report, "EnvPATH: %s\n", env_p);

  I then installed the crash-in-postinst package from the daisy-plucker-
  seeds PPA and received a crash report with the EnvPATH key and value.
  Its worth noting that when apport gathers environment information only
  a white listed set of keys is gathered to prevent disclosing
  potentially sensitive information. So we should look at what
  environment variables the dist-upgrader sets and gather those at a
  minimum.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1825021/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp


[Touch-packages] [Bug 1825021] Re: apt's dpkgpm.cc WriteApportReport function should gather more data

2019-04-16 Thread Julian Andres Klode
FWIW, apt actually overrides PATH since bionic or cosmic or something
like that.

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apt in Ubuntu.
https://bugs.launchpad.net/bugs/1825021

Title:
  apt's dpkgpm.cc WriteApportReport function should gather more data

Status in apt package in Ubuntu:
  New

Bug description:
  While fixing bug 1813464 it occurred to me that it is not obvious that
  the dist-upgrader tarball includes its own copy of invoke-rc.d and
  that this has been complicated to debug more than once. The apport
  information in that report was gathered by apt's dpkgpm.cc code and I
  think we should extend that to gather additional information from the
  environment at the time that the package installation failed. For
  example the dist-upgrader sets the following:

  os.environ["PATH"] = "%s:%s" % (os.getcwd()+"/imported",
  os.environ["PATH"])

  Gathering path from the environment would have made diagnosing bug
  1813464 a lot easier. I tested modifying dpkgpm.cc with the following:

 if(const char* env_p = std::getenv("PATH"))
fprintf(report, "EnvPATH: %s\n", env_p);

  I then installed the crash-in-postinst package from the daisy-plucker-
  seeds PPA and received a crash report with the EnvPATH key and value.
  Its worth noting that when apport gathers environment information only
  a white listed set of keys is gathered to prevent disclosing
  potentially sensitive information. So we should look at what
  environment variables the dist-upgrader sets and gather those at a
  minimum.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1825021/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp


[Touch-packages] [Bug 1825021] Re: apt's dpkgpm.cc WriteApportReport function should gather more data

2019-04-16 Thread Brian Murray
In the event that this were SRU'ed for it to be useful we'd need to SRU
apt to the release from which people were upgrading.

** Tags added: disco rls-ee-incoming

** Changed in: apt (Ubuntu)
   Importance: Undecided => High

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apt in Ubuntu.
https://bugs.launchpad.net/bugs/1825021

Title:
  apt's dpkgpm.cc WriteApportReport function should gather more data

Status in apt package in Ubuntu:
  New

Bug description:
  While fixing bug 1813464 it occurred to me that it is not obvious that
  the dist-upgrader tarball includes its own copy of invoke-rc.d and
  that this has been complicated to debug more than once. The apport
  information in that report was gathered by apt's dpkgpm.cc code and I
  think we should extend that to gather additional information from the
  environment at the time that the package installation failed. For
  example the dist-upgrader sets the following:

  os.environ["PATH"] = "%s:%s" % (os.getcwd()+"/imported",
  os.environ["PATH"])

  Gathering path from the environment would have made diagnosing bug
  1813464 a lot easier. I tested modifying dpkgpm.cc with the following:

 if(const char* env_p = std::getenv("PATH"))
fprintf(report, "EnvPATH: %s\n", env_p);

  I then installed the crash-in-postinst package from the daisy-plucker-
  seeds PPA and received a crash report with the EnvPATH key and value.
  Its worth noting that when apport gathers environment information only
  a white listed set of keys is gathered to prevent disclosing
  potentially sensitive information. So we should look at what
  environment variables the dist-upgrader sets and gather those at a
  minimum.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1825021/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp