ALT Linux sysadmins discussion
 help / color / mirror / Atom feed
From: Michael Shigorin <mike@osdn.org.ua>
To: sysadmins@lists.altlinux.org
Subject: [Sysadmins] Fwd: [monit-dev] Re: Proposal: remove redirect support in http protocol test
Date: Mon, 15 Oct 2007 18:47:30 +0300
Message-ID: <20071015154730.GX13827@osdn.org.ua> (raw)

	Здравствуйте.
Коллеги, кто-нибудь ещё пользуется тестом HTTP в monit?
Апстрим думает, стоит ли держать поддержку редиректов
или выкинуть как излишнее (и несовершенное) усложнение.

----- Forwarded message from Martin Pala <martinp/tildeslash.com> -----

Date: Thu, 11 Oct 2007 21:31:13 +0200
From: Martin Pala <martinp/tildeslash.com>
To: The monit developer list <monit-dev/nongnu.org>
Subject: [monit-dev] Re: Proposal: remove redirect support in http protocol test

Good point ...  on the other side, there can be users already who use 
this test and thus will need to modify the configuration on upgrade.

The redirect test in monit cvs can also identify the loops, which can be 
useful.

So here is my +-0 (the proposed change has positive and negative features)

Martin


Jan-Henrik Haukeland wrote:
>The HTTP protocol test supports rudimentary redirect within the same 
>server but not to external servers, though that should not be difficult 
>to add. However looking at the redirect code now I think the best would 
>be to remove it from monit. The reason is that it adds extra unnecessary 
>complexity without adding much in functionality and it also keeps the 
>socket open for keep-alive timeout. Users should instead write the 
>direct URL and we should send the "Connection: close" header as well 
>which would be easier on the server we test. For monit it does not 
>matter though since the socket is closed anyway after the http protocol 
>test returns.
>
>Jan-Henrik

----- End forwarded message -----

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/
 ----       Oct 26--27, Kiev, Ukraine:
--       http://conference.osdn.org.ua


                 reply	other threads:[~2007-10-15 15:47 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20071015154730.GX13827@osdn.org.ua \
    --to=mike@osdn.org.ua \
    --cc=shigorin@gmail.com \
    --cc=sysadmins@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux sysadmins discussion

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/sysadmins/0 sysadmins/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 sysadmins sysadmins/ http://lore.altlinux.org/sysadmins \
		sysadmins@lists.altlinux.org sysadmins@lists.altlinux.ru sysadmins@lists.altlinux.com
	public-inbox-index sysadmins

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.sysadmins


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git