[Bug 1953572] Re: Broken links in the View Trends and the View Histogram menu

[Mauricio Faria de Oliveira]

Thanks, Lena!

I'm not biased towards including or not including the non-code changes (eg, Changelog and THANKS), 
as I'd presume any potential problems in future updates due to these files should happen because
the future patches would be touching these files as well. This would mean the future patch didn't
remove them either, and if it didn't, maybe having these changes introduced now would help keeping
these files more up-to-date, so that future patches apply without changes.
(Not sure I'm missing something in maintenance context that you're familiar with and I'm not -- happy to learn more!)

Yeah, the LP bug number without parentheses caught my eyes too, but
AFAIK, the LP automation only strictly requires the `LP: #<number>` part
(seen other SRUs like that). The parentheses are easier on the eye /
more as usual too.

I'm happy to change that in the stable releases too.

If you don't mind the Changelog/THANKS changes, please feel free to
proceed with the upload!

Thanks again for your help :)

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1953572

Title:
  Broken links in the View Trends and the View Histogram menu

Status in nagios4 package in Ubuntu:
  In Progress
Status in nagios4 source package in Focal:
  In Progress
Status in nagios4 source package in Jammy:
  In Progress
Status in nagios4 source package in Mantic:
  Won't Fix
Status in nagios4 source package in Noble:
  In Progress
Status in nagios4 source package in Oracular:
  In Progress

Bug description:
  [Impact]
  The links for the following are broken in the Service Information or the Host Information menus for the following links:

  View Trends For This Service
  View Alert Histogram For This Service
  View Trends For This Host
  View Alert Histogram For This Host

  [Test Plan]
  See deployment steps in comment #15.
  Just access those links in: Current status / Hosts and Current status / Services and choosing a host or a service
  Test all other links following the menu/pages for no broken links / regressions.

  [Where problems could occur]
  The URLs for the affected pages have been changed from relative to absolute. Nothing else has changed so this patch should not affect any other page. The worst case scenario would be that the patch has inadvertently modified the link for other pages. In any case the absolute links should also work.

  [Other Info]
  Upstream code has been patched to fix a bug that required that the html files (on /nagios4) should be one folder level below cgi-bin. This was because sometimes Nagios tried to reach html files as ../file.html while the base url was pointing to the cgi-bin folder. This relative URLs where substituted for absolute URLs. The deb package sets the cgi url to /cgi-bin/nagios4 so was affected by this issue.

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