Load message body on demand

[andreasohlund]

SC now supports loading of message bodies

Particular/ServiceControl@2b7a6d9

This is the way it works. When querying for messages there will by a property called "body_url" returned:

body_url: "http://localhost:33333/api/messages/c143e4ee-1e35-827c-ded9-73d4a551563a/body"

To get the body just download the url as bytes eg:

using (var client = new WebClient())
{
     body = client.DownloadData(message.BodyUrl);
}

This will stream back the contents of the body. The returned content type will reflect that actual content type of the body: (XML in the below case)

Access-Control-Allow-Headers:Origin, X-Requested-With, Content-Type, Accept
Access-Control-Allow-Methods:POST, GET, PUT, DELETE, OPTIONS
Access-Control-Allow-Origin:*
Access-Control-Expose-Headers:ETag, Last-Modified, Link, Total-Count, X-Particular-Version
Cache-Control:private, max-age=0
Content-Encoding:gzip
Content-Type:text/xml
Date:Wed, 18 Dec 2013 20:05:26 GMT
Expires:Thu, 18 Dec 2014 20:05:26 GMT
Server:Microsoft-HTTPAPI/2.0
Transfer-Encoding:chunked
X-Particular-Version:0.8.0-unstable294

[dannycohen]

content-length ?

[andreasohlund]

done

On Wed, Dec 18, 2013 at 9:37 PM, Danny Cohen notifications@github.comwrote:

content-length ?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/185#issuecomment-30878146
.

[dannycohen]

do you return HTTP 413 "Request Entity Too Large" on certain size and larger ?

[andreasohlund]

No,add a backlog issue for that

On Wed, Dec 18, 2013 at 10:26 PM, Danny Cohen notifications@github.comwrote:

do you return HTTP 413 "Request Entity Too Large" on certain size and
larger ?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/185#issuecomment-30882318
.

[dannycohen]

NP. Will do.

[dannycohen]

@andreasohlund - Have you added the message size to the message data ? (as we discussed, in order to allow SI to decide whether to display the message or consider it as too large for display and only allow downloading it)

[andreasohlund]

Yes, BodySize is returned

Sent from my iPhone

On 25 dec 2013, at 10:33, Danny Cohen notifications@github.com wrote:

@andreasohlund - Have you added the message size to the message data ? (as we discussed, in order to allow SI to decide whether to display the message or consider it as too large for display and only allow downloading it)

—
Reply to this email directly or view it on GitHub.

[HEskandari]

@dannycohen Please see if the implementation is satisfactory.

[dannycohen]

@HEskandari -
See @andreasohlund comment - 314e024#commitcomment-4949120

It makes sense to do the loading when clicking on the Body tab and not when changing selected messages.

[HEskandari]

@dannycohen there are a couple of isues there so I took a short cut. Main one being, what should happen if the body tab is already selected and then you click a message? Because of how the tab will get selected - select the body tab and restart the app, it will get selected / focused on application start - we don't get a notification about the tab getting the focus when a message is selected.