On Tue, Mar 11, 2014 at 6:28 PM, Klaus Trainer <notifications@github.com>wrote:

> In src/couch_replicator/src/couch_replicator_httpc.erl:
>
> > +    true -> start_new_session(HttpDb, Worker)
> > +    end.
> > +
> > +
> > +need_new_session(#httpdb{credentials = undefined}) ->
> > +    false;
> > +
> > +need_new_session(#httpdb{credentials = Credentials}) ->
> > +    case ets:lookup(Credentials, cookie) of
> > +    [] ->
> > +        true;
> > +    [{cookie, _, UpdatedAt}] ->
> > +        %% As we don't know when the cookie will expire, we just decide
> > +        %% that we want a new session if the current one is older than
> > +        %% one minute.
> > +        OneMinute = 60 * 1000000, % microseconds
>
> It looks like an expiry time is only set if the
> "allow_persistent_cookies" configuration is set to true, which is
> false by default, however. See the max_age/0 function in the
> couch_httpd_auth module.
>

yeah we are sending a wrong header imo.

- benoit

>  --
> Reply to this email directly or view it on GitHub<https://github.com/apache/couchdb/pull/172/files#r10485167>
> .
>