Comment on revision r1150 in mortar

0 views
Skip to first unread message

mor...@googlecode.com

unread,
Dec 12, 2009, 3:54:03 AM12/12/09
to mortar-de...@googlegroups.com
ted...@tedivm.com commented on revision r1150 in project mortar.
Details are at http://code.google.com/p/mortar/source/detail?r=1150

General Comment:
This closes issue #1

Respond to these comments at
http://code.google.com/p/mortar/source/detail?r=1150
--
You received this message because you starred this review, or because
your project has directed all notifications to a mailing list that you
subscribe to.
You may adjust your review notification preferences at:
http://code.google.com/hosting/settings

tedivm

unread,
Dec 12, 2009, 4:12:59 AM12/12/09
to mortar-de...@googlegroups.com
Oh shiny, I didn't realize I set up the comments on commits to do to this list. That kind of makes sense.

Anyways I finally killed the first bug on the list (issue #1), which is about how we handle outputting errors across different formats and io processors (like standard http, rest, cli).

Basically, the system used to error out if someone tried to access an image or icon (favicon.ico for instance) instead of giving one of our standard error pages. This would occur for any format not explicitly handled.

Now each handler gets to specify an error format for all its requests. Standard web stuff will return html pages, while the REST web stuff will return rss (its default format), and command line actions (including cron) will send out errors as text.

Rpbert
> You received this message because you are subscribed to the Google Groups "Mortar Development" group.
> To post to this group, send email to mortar-de...@googlegroups.com.
> To unsubscribe from this group, send email to mortar-developm...@googlegroups.com.
> For more options, visit this group at http://groups.google.com/group/mortar-development?hl=en.
>
>

tedivm

unread,
Dec 12, 2009, 4:13:51 AM12/12/09
to mortar-de...@googlegroups.com
Wow, I can't believe I misspelled my own name . . .

Robert
Reply all
Reply to author
Forward
0 new messages