[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-ha-dev
Subject:    Re: [Linux-ha-dev] [PATCH]Monitor failure and IPv6 support of apache-ra
From:       Dejan Muhamedagic <dejan () suse ! de>
Date:       2012-01-31 13:02:28
Message-ID: 20120131130224.GE28262 () walrus ! homenet
[Download RAW message or body]

Hi Keisuke-san,

On Tue, Jan 31, 2012 at 09:52:24PM +0900, Keisuke MORI wrote:
> Hi Dejan
> 
> 2012/1/31 Dejan Muhamedagic <dejan@suse.de>:
> > Hi Keisuke-san,
> (...)
> > On Tue, Jan 31, 2012 at 08:46:35PM +0900, Keisuke MORI wrote:
> >> The current RA will try to check the top page (http://localhost:80)
> >> as the default behavior if you have not enabled server-status in httpd.conf
> >> and it would fail to start even for the apache's default test page:)
> >
> > Hmm, the current RA would produce an error for that URL:
> >
> > 488     case $STATUSURL in
> > 489         http://*/*) ;;
> > 490         *)
> > 491         ocf_log err "Invalid STATUSURL $STATUSURL"
> > 492         exit $OCF_ERR_ARGS ;;
> > 493     esac
> 
> Strange. That URL is generated by the RA itself.
> 
> apache-conf.sh:
>    119  buildlocalurl() {
>    120    [ "x$Listen" != "x" ] &&
>    121          echo "http://${Listen}" ||
>    122          echo "${LOCALHOST}:${PORT}"
> 
> 
> Probably we should relax the validation pattern, as just 'http://*' ?

Agreed. I thought that the intention was to always use the status
page, but obviously people figured out that they could skip that.
Just as well.

[...]

Cheers,

Dejan
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
[prev in list] [next in list] [prev in thread] [next in thread]