On 5/26/09 4:20 AM, Wichert Akkerman wrote: >> from repoze.bfg.security import logout >> >> def someview(context, request): >> response = render_template_to_response('some.pt') >> logout(response) >> return response >> >> vs. >> >> from repoze.bfg.security import forget >> >> def someview(context, request): >> response = render_template_to_response('some.pt') >> headers = forget() >> response.headerlist.extend(headers) >> return response > > The first variant feels better. I can not see a use case where you would > ever want to manipulate the headers returned by logout()/remember() > before adding them to the response, so folding the response header > updating into the function itself makes sense. Saves one often > duplicated line of code.
OK. If nothing earth-shattering pops up, I'll add "login" and "logout" convenience functions. - C _______________________________________________ Repoze-dev mailing list Repoze-dev@lists.repoze.org http://lists.repoze.org/listinfo/repoze-dev