Bug 853018 - Templates batch cleanup: rel links, "forgotten password" feature, consistent titles
Summary: Templates batch cleanup: rel links, "forgotten password" feature, consistent ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: conga
Version: 5.9
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: rc
: ---
Assignee: Jan Pokorný [poki]
QA Contact: Cluster QE
URL:
Whiteboard:
Depends On: 514679
Blocks: 969202
TreeView+ depends on / blocked
 
Reported: 2012-08-30 09:35 UTC by Radek Steiger
Modified: 2013-10-01 00:39 UTC (History)
4 users (show)

Fixed In Version: conga-0.12.2-67.el5
Doc Type: Bug Fix
Doc Text:
Proposed text: Prior to this update, luci, the web-based frontend of conga cluster management, contained non-visual links dedicated to better browsing experience in the agents supporting it, such as a navigation to the a site map, access of which resulted in the error. Another minor inconvenience was a rare inconsistency in the page titles. The luci templates cleanup was done to fix respective corner cases.
Clone Of:
: 969202 (view as bug list)
Environment:
Last Closed: 2013-10-01 00:39:58 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
logs (10.89 KB, text/x-log)
2012-08-30 09:37 UTC, Radek Steiger
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2013:1358 0 normal SHIPPED_LIVE conga bug fix update 2013-09-30 21:12:28 UTC

Description Radek Steiger 2012-08-30 09:35:51 UTC
Description of problem:
Luci page header provides rel links to 'search_form', 'author' and 'sitemap' which do not exist and result in portal error message and non-fatal tracebacks when opened.


Version-Release number of selected component (if applicable):
luci-0.12.2-64.el5


How reproducible:
Always


Steps to Reproduce:
1. Check the rel links provides in luci page header
2. Try to open any of the last three


Actual results:
No content and traceback.


Expected results:
The pages should not traceback or not exist at all.


Additional info:
The rel links provided in the page source:
<link rel="search" href="https://mycluster.example.com:8084/luci/search_form" title="Search this site" />
<link rel="author" href="https://mycluster.example.com:8084/luci/author/" title="Author information" />
<link rel="contents" href="https://mycluster.example.com:8084/luci/sitemap" title="Site Map" />

Comment 1 Radek Steiger 2012-08-30 09:37:26 UTC
Created attachment 608100 [details]
logs

Comment 2 Jan Pokorný [poki] 2012-08-30 12:38:27 UTC
Thanks, Radek.

In fact, there is more places like this, but generally this technical
debt regarding templates is very minor.  I prefer solving this bug
(as batch of templates editting) together with any other requiring
to modify Data.fs.  For the time being, this bug can be used to track
all these tiny things that will show up.

In parallel, some of them are tracked in luci/TODO [1], such
as completely disabling the "forgotten password" feature,
which equally well cannot be triggered in a standard-user mode
(as opposed to nit-picking-tech-savvy one).

[1] http://git.fedorahosted.org/cgit/conga-luci-1stgen.git/tree/?h=RHEL5-active

Comment 3 Jan Pokorný [poki] 2012-09-06 17:27:43 UTC
Attaching [bug 514679], credit for discovery belongs to Radek (who
unfortunately didn't look into conga's bug history to find it reported
ages ago).

Comment 4 Jan Pokorný [poki] 2012-11-09 17:56:29 UTC
Another thing: there is (AFAIK) JS-based title rewriting that works
everywhere except for "cluster" tab, making it inconsistent.

Comment 5 Jan Pokorný [poki] 2012-11-09 19:39:57 UTC
The next one is that every explicit logout is accompanied with

> 2012-11-09T20:26:47 INFO CMFFormController
> You have triggered the form controller action "logout" using
> a GET REQUEST.  This is a potential security hazard.
> In Plone 3.0 this will FAIL unless you explicitly enable your form
> to support GET requests in the ZMI (or using the .metadata file).

in /var/lib/luci/log/event.log.

This could be prevent by turning "log out" link into trigger of
a POST form with hidden ":default_method=logout" parameter.

Comment 7 Jan Pokorný [poki] 2013-05-28 09:27:23 UTC
re: [comment 4]:
This will be kept, only non-rewritten titles will change from:

    <page title> &mdash; <portal title>

to

    <Capitalized portal title> &mdash; <page title>

so that it conforms to the format in rewritten titles (i.e., no harassing
inconsistence).

Comment 8 Jan Pokorný [poki] 2013-05-28 10:02:57 UTC
re [comment 5]:

This seems to be outside our scope, it's rather an internal Plone
inconsistency (or, less likely, leftover from historic updates).

The action handler is declated in:
> CMFPlone/profiles/default/actions.xml

Comment 10 Jan Pokorný [poki] 2013-05-28 12:22:36 UTC
Recap:

> - non-existent pages linked from page head section ('search_form', 'author'
>   and 'sitemap')

Radek knows best :)  The only <link> tag kept is the one referring to the
main page (should point to /luci), as it indeed exists.

> - disabled sending recovery email in (failsafe_)login_form completely

IIRC, was possible when logged in and accessing /luci/require_login,
/luci/login_form or something like that.  Will look at more if cannot be
located.  Originally, it lead to non-servable address.

> - JS-based title rewriting in cluster tab

It would be swimming against the stream, so I instead made JS-based and
native titles more consistent: see [comment 4].  Visible change at
/luci/homebase, luci/storage (native titles) -- now they should conform
to /luci/cluster, /luci/homebase?pagetype=2 and others where JS rewriting
applies (basically all three-items-titles like "Luci - foo - bar").

> - logout action using more appropriate request method

More like internal Plone implementation/integration issue, see [comment 8]
and [comment 9].  Not addressed, nor a big deal with that.

Comment 16 errata-xmlrpc 2013-10-01 00:39:58 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2013-1358.html


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