Your message dated Mon, 12 Feb 2024 17:50:00 +0000
with message-id <[email protected]>
and subject line Bug#990989: fixed in xymon 4.3.30-2
has caused the Debian Bug report #990989,
regarding xymon: report.sh and snapshot.sh fail with firefox
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
990989: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=990989
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Source: xymon
Version: 4.3.30-1
Severity: normal

Dear Maintainer,

I noticed, that in Xymon 4.3.28 as well as in 4.3.30 opening the
"Snapshot Report" (snapshot.sh) as well as the "Availability Report"
(report.sh) in the "Reports" menu do not work as expected, but try to
download a file snapshot.sh or report.sh.

Digging deeper into this, I noticed, that this seems a Firefox
(currently I use 89.0.2) only problem, while chromium works without
problems.

In the code web/snapshot.c and web/report.c I found a variable
usemultipart, which is 1 by default and is reset to 0 if
HTTP_USER_AGENT contains "KHTML", which (according a comment) cannot
handle multipart documents.

It seems, that current Firefox also can not handle this correct, while
Chromium uses the User-Agent
"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like
Gecko) Chrome/90.0.4430.212 Safari/537.36"#
so it already disables multipart.

From my point of view, it may be a good idea to disable usemultipart
by default, if Safari, Konqueror and Chrom(ium) disable it via the
KHTML in the User-Agent and Firefox doesn't work with usemultipart=0.

A patch against 4.3.28-5+deb10u1 is attached, which solves the
problems for me using Firefox, it should also wirk with 4.3.30.

Greetings
Roland
--- a/web/report.c
+++ b/web/report.c
@@ -201,7 +201,7 @@ int main(int argc, char *argv[])
 	int argi, newargi;
 	char *envarea = NULL;
 	char *useragent = NULL;
-	int usemultipart = 1;
+	int usemultipart = 0;
 
 	newargi = 0;
 	xymongen_argv[newargi++] = xymongencmd;
--- a/web/snapshot.c
+++ b/web/snapshot.c
@@ -145,7 +145,7 @@ int main(int argc, char *argv[])
 	int argi, newargi;
 	char *envarea = NULL;
 	char *useragent;
-	int usemultipart = 1;
+	int usemultipart = 0;
 
 	newargi = 0;
 	xymongen_argv[newargi++] = xymongencmd;

Attachment: signature.asc
Description: PGP signature


--- End Message ---
--- Begin Message ---
Source: xymon
Source-Version: 4.3.30-2
Done: Roland Rosenfeld <[email protected]>

We believe that the bug you reported is fixed in the latest version of
xymon, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Roland Rosenfeld <[email protected]> (supplier of updated xymon package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Mon, 12 Feb 2024 18:05:54 +0100
Source: xymon
Architecture: source
Version: 4.3.30-2
Distribution: unstable
Urgency: medium
Maintainer: Axel Beckert <[email protected]>
Changed-By: Roland Rosenfeld <[email protected]>
Closes: 646168 903614 930532 990989 999921 1017462 1028005 1039429 1057044
Changes:
 xymon (4.3.30-2) unstable; urgency=medium
 .
   [ Axel Beckert ]
   * Bump debhelper-compat to 13.
     + Override dh_missing with --list-missing due to #968170 in
       dh_apache2/apache2-dev.
   * Lintian Overrides:
     + Update where needed to match current phrasing of the tags.
     + Add new overrides for overzealous spare-manual-page and
       package-contains-documentation-outside-usr-share-doc lintian tags.
     + Remove lintian override for no more emitted
       apache2-deprecated-auth-config tag.
     + Add overrides for very-long-line-length-in-source-file.
   * Bump debian/watch version to 4.
   * Remove packaging adaptions for the hobbit to xymon migration as well
     as maintainer-script snippets caring about versions before 4.3.7-1~~.
   * Drop /var/log/xymon/ and /var/lib/xymon/ directories from being
     shipped in the package. They're created in postinst anyways if they
     don't exist. And they need to be chowned to the dynamically created
     xymon user in postinst no matter what.
   * Declare compliance with Debian Policy 4.6.0. (No changes needed.)
 .
   [ Debian Janitor ]
   * Remove constraints unnecessary since buster:
     + Build-Depends: Drop versioned constraint on dpkg-dev, libldap2-dev,
       libpcre3-dev and libssl-dev.
     + xymon: Drop versioned constraint on lsb-base and xymon-client in Depends.
     + xymon: Drop versioned constraint on hobbit-plugins in Breaks.
     + xymon-client: Drop versioned constraint on debconf in Pre-Depends.
     + xymon-client: Drop versioned constraint on hobbit-plugins and xymon in
       Breaks.
     + Remove 77 maintscript entries from 2 files.
 .
   [ Carsten Leonhardt ]
   * Update xymon-client.init: add "reload" to usage message as that is
     supported, too.
 .
   [ Yavor Doganov ]
   * debian/patches/93_pcre2.patch: New; port to PCRE2 (Closes: #999921).
   * debian/control (Build-Depends): Replace libpcre3-dev with libpcre2-dev.
   * debian/rules (override_dh_auto_configure): Use pcre2-config.
 .
   [ Roland Rosenfeld ]
   * 94_nousemultipart: set usemultipart=0 in snapshot and report to fix
     Firefox (Closes: #990989).
   * 95_fix_tls13_banner: Fix problems with imaps/pop3s (Closes: #930532).
   * Replace dependency on lsb-base lsb-release.
   * Update lintian overrides to new syntax.
   * Update debian/po/es.po.  Thanks to Camaleón (Closes: #1017462).
   * Update debian/po/pt_BR.po.  Paulo Henrique de Lima Santana
     (Closes: #1028005).
   * Declare compliance with Debian Policy 4.6.2 (No changes needed).
   * Add debian/salsa-ci.yml pipeline (with crossbuild-arm64 disabled).
   * Remove 87_fix_logfetch_FTBFS_with_glibc_2.26, which is included
     upstream since 4.3.29~r8060.
   * Add a systemd xymon-client.service file.
   * Add a systemd xymon.service file (Closes: #1039429).
   * Move xymond_client from xymon to xymon-client (Closes: #903614)
   * Remove invoke-rc.d hobbit[-client] stop calls from preinst scripts.
   * Auto-generate apache2 recommends via ${misc:Recommends}.
   * Convert CREDITS to UTF-8.
   * Remove obsolete Homepage field from d/u/metadata.
   * Lintian override apache2-deprecated-auth-config, because offending
     lines are already wrapped by <IfModule !mod_authz_core.c>.
   * 96_manual-backslash: Correct encoding of backslash in man pages.
   * Regenerate HTML documentation during build (Closes: #646168).
   * 97_makehtml: Makes HTML documentation reproducible.
   * 21_FHS-instead-FSSTND-in-example-in-man-page: remove html patch, since
     this is now auto generated.
   * 98_man_fb: Fix typo \fb (instead of \fB) in some man pages.
   * 92_ntpdate_ntpdig: Use "ntpdig -t 1" (from ntpsec-ntpdig) instead of
     "ntpdate -u -q -p 1" (shell wrapper) (Closes: #1057044).
   * 93_pcre2: Fix pcre2_substring_copy_bynumber() return values.
   * Exchange Maintainer and Uploaders, and add myself to Uploaders.
   * Apply wrap-and-sort -a.
Checksums-Sha1:
 4af3de5d40f1af489697106ee62afd794207ffc1 2068 xymon_4.3.30-2.dsc
 bceb621cdd7032344a1e145bbe8fd8047dc52b23 59268 xymon_4.3.30-2.debian.tar.xz
 2377a83d71fc7a815277164daa853619a56dc6c3 9471 xymon_4.3.30-2_source.buildinfo
Checksums-Sha256:
 7f4835e6bf4920a7d74e8590fc435912c15d82c5a25621e0dd528f21d72e4aa4 2068 
xymon_4.3.30-2.dsc
 738535fb85f24e6651a186409cb24cbe645cf3ceaede3cfe64f044653a043069 59268 
xymon_4.3.30-2.debian.tar.xz
 39eef56eed7a6ff7292ecac91cd5f2537ba6d654563ceb8d47e3ce3742756df6 9471 
xymon_4.3.30-2_source.buildinfo
Files:
 29880f218c7d51a89cd2c713ff0a3c1a 2068 net optional xymon_4.3.30-2.dsc
 965167ba40afa1d41669f2be00f96328 59268 net optional 
xymon_4.3.30-2.debian.tar.xz
 a0b7ef2ecb3fe86b51d058344bd3a5d0 9471 net optional 
xymon_4.3.30-2_source.buildinfo

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEErC+9sQSUPYpEoCEdAnE7z8pUELIFAmXKUEgACgkQAnE7z8pU
ELJ/hhAAtJSKoNWtmhMYUniIyqSd13jy32Jl4U2Ft5LBhBCip3CRo4lrwWPKqFXv
47nnFY9OyzXWxjOa7qTMpz3EY+yQimygUsmtoLOdS53IFZcrJmuERZJUs/T7OCvP
O0aYtUezK6a0npunLdoWJgrd9gRiEy2wkxoa2+AKiGu6FamwPC02OPcQ6zeUrMkb
zk7rwWydpah7LT8+OFR6rXPZcw1PKQOhuOBkNW+o2t9rFVbgBORlTxM2PBRSMKIg
/JS9lBufCO7gVsMLQ2lvQAHou0v4XGncugyhzgKjHDBpvoYIL/RuSsMqqqIYDLPB
2HzmtFwpv65KIRuCLQbT2uAjYsrarrh4+ZWaVQCSHDmVL6GlNYz5zfRyZMNdbqsL
3WnUoN6XlVEiBK8d2xR+MZm8HU+zrCqgductO96XSdDxjNk8deORa35UV2+t/vXT
ZrYQUjqgWQWXNsaGUbudSCx+hY4yuf9NujPzwaMf/8I/YzkR1M65oNp8qoFUGlP0
I2R/27/iJi37wFja3CfEJ2F08SCMlrvO6DaCWzCkNg5tvioIwxLL1ZVkSDoSjZOB
zn7a29nF46vWkipV7luGm1F6vII4/IZoHNL5jDSGPgkj/pfraBXox1RbyFECvnH7
CKQHdIMCmML6to2BM+DO+MVvQbyHHl0zvSfrIPkro9mIYlZO5/Y=
=5qLl
-----END PGP SIGNATURE-----

Attachment: pgplsUDvZeJkU.pgp
Description: PGP signature


--- End Message ---

Reply via email to