[haiku-commits] Re: haiku: hrev50905 - in src/apps/haikudepot: server model tar util .
- From: Andrew Lindesay <apl@xxxxxxxxxxxxxx>
- To: haiku-commits@xxxxxxxxxxxxx
- Date: Tue, 31 Jan 2017 21:24:24 +1300
Hello Adrien;
Some comments below about usage of the HTTP classes. Mostly my fault for
still not having written the documentation for them.
Thank you for those tips; I've made most of those changes now.
+/* The output format for this is suitable for use in the
"If-Modified-Since"
+ * header. An example of this output would be;
+ * 'Fri, 24 Oct 2014 19:32:27 +0000'
+ */
I think this is already implemented in BHttpDate.
I wasn't able to find that class in the system. Can you let me know
specifically where that might be and I'll take another look.
+const BString
+ServerSettings::_GetUserAgentVersionString()
Should we make this the default behavior for the HTTP user agent? (create
something from the app info name and version?)
That might be a good idea generally. In this case, I may need to be
more specific with the "User-Agent" header because I need to be able to
detect incompatible clients in the future so may need to retain specific
control over it.
Regards;
--
Andrew Lindesay
Other related posts: