Thanks for the branch Jimmy. I think you make some good points about enabling some form of authentication in the REST server. I've looked at:
http://ish.io/embedded/restish/guard.html which provides examples of hooking up repoze.who. Did you think about that and if so, why did you choose not to use it (adding a dependency and the extra code complexity is a valid answer :). Have you thought about using something like OAuth? Are you concerned at all about cleartext passwords? Finally, while your patch looks basically decent (I'd quibble with some whitespace, but that's unimportant), please consider adding a test and/or some documentation (the latter perhaps as a doctest). And are you willing and able to assign your copyright to the FSF? -- https://code.launchpad.net/~jimmy-sigint/mailman/restapi_auth/+merge/36833 Your team Mailman Coders is requested to review the proposed merge of lp:~jimmy-sigint/mailman/restapi_auth into lp:mailman. _______________________________________________ Mailman-coders mailing list [email protected] http://mail.python.org/mailman/listinfo/mailman-coders
