aboutsummaryrefslogtreecommitdiff
path: root/web/module
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2015-04-23 18:46:03 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2015-04-23 18:46:03 +0200
commitb4c7f615916cd41e4772c5dc1b2f9f59b3b2ae6c (patch)
treeb0eb5b444be82073b21cef51c6d2ee5fc2e947ae /web/module
parent4d52ce80abf0e31ec42b87033d06e7055866b68b (diff)
Additional clarifications
Diffstat (limited to 'web/module')
-rw-r--r--web/module18
1 files changed, 9 insertions, 9 deletions
diff --git a/web/module b/web/module
index 5e5873d..3e97ff7 100644
--- a/web/module
+++ b/web/module
@@ -85,13 +85,10 @@ namespace web
virtual const name_values&
cookies () = 0;
- // Get the stream to read the request content from.
- //
- // Throw sequence_error if some unbuffered content is already written.
- //
- // @@ Why can't I write unbuffered content while reading
- // the request? Is it so? If so, is it implementation
- // details?
+ // Get the stream to read the request content from. Note that
+ // reading content after any unbuffered content has been written
+ // is undefined behavior. The implementation may detect it and
+ // throw sequence_error but is not required to do so.
//
virtual std::istream&
content () = 0;
@@ -168,10 +165,13 @@ namespace web
public:
// Any exception other than invalid_request described above that
// leaves this function is treated by the web server implementation
- // as an internal server error (505). Similar to invalid_request,
+ // as an internal server error (500). Similar to invalid_request,
// it will try to return the status and description (obtained by
// calling what() on std::exception) to the client, if possible.
- // The description is assume to be encoded in UTF-8.
+ // The description is assume to be encoded in UTF-8. The
+ // implementation may provide a configuration option to omit
+ // the description from the response, for security/privacy
+ // reasons.
//
virtual void
handle (request&, response&, log&) = 0;