# [PATCH] fix for bug #31455 (multiple session_start() creates multiple session cookies in HTTP-Response)

- Thread: https://externals.io/message/14876
- Original archive: https://news-web.php.net/php.internals/14876
- Started by Antony Dovgal on 2005-02-12T17:13:23+00:00
- Messages: 3
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #14876 — Antony Dovgal — 2005-02-12T17:13:23+00:00

URL: https://externals.io/message/14876#14876

Hi all!

Does anyone object to this fix merged into all branches ?

http://tony2001.phpclub.net/dev/tmp/bug31455.diff

-- 
Wbr, 
Antony Dovgal aka tony2001

---

## #14882 — Jani Taskinen — 2005-02-12T18:03:37+00:00

URL: https://externals.io/message/14876#14882 · In reply to: #14876

> Does anyone object to this fix merged into all branches ?
>
> http://tony2001.phpclub.net/dev/tmp/bug31455.diff
>

     What's the difference with PS(cookie_send) and PS(sent_cookie) ??

     --Jani

---

## #14885 — Antony Dovgal — 2005-02-12T20:44:26+00:00

URL: https://externals.io/message/14876#14885 · In reply to: #14882

On Sat, 12 Feb 2005 20:03:37 +0200 (EET)
Jani Taskinen <sniper@iki.fi> wrote:

> 
> 
> > […]
> 
>      What's the difference with PS(cookie_send) and PS(sent_cookie) ??

PS(send_cookie) is used to determine if we need to set cookie when client
didn't send it to the server.
PS(cookie_sent) it turn is required to eliminate miltiple cookies to be set.

Jani, I know that ext/session is a collection of hacks.
But IMO this doesn't mean that we should not fix bugs there (and yes, I think 
the current behaviour looks more like a bug).

This doesn't mean that I'm going to commit the fix if you do strongly
object, though =)

-- 
Wbr, 
Antony Dovgal aka tony2001
