Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of simon.macdonald@gmail.com
 designates 209.85.160.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAM2q_UxTsA99bWpp+8jkszhP_+qJ8=ZoKwxRcRwBfZGz8yWd9w@mail.gmail.com>
References: 
 <CAM2q_UxTsA99bWpp+8jkszhP_+qJ8=ZoKwxRcRwBfZGz8yWd9w@mail.gmail.com>
From: Simon MacDonald <simon.macdonald@gmail.com>
Date: Mon, 14 May 2012 14:50:50 -0400
Message-ID: 
 <CAM1AYQC62Jn_yYdHY-W=GNpgBbHrKa0uRPDtyJo00-e1kWUubg@mail.gmail.com>
Subject: Re: Adding HTTP status code to FileTransferError
To: callback-dev@incubator.apache.org
Content-Type: multipart/alternative; boundary=047d7b2ed963cae74e04c00391bb

--047d7b2ed963cae74e04c00391bb
Content-Type: text/plain; charset=ISO-8859-1

Sounds good to me. I'd probably skip the "http_message" as you pretty much
get all you need from the status and you don't need to localize the string.

Simon Mac Donald
http://hi.im/simonmacdonald


On Mon, May 14, 2012 at 9:29 AM, Don Coleman <don.coleman@gmail.com> wrote:

> I'd like to add HTTP Status Code and HTTP Status Message to
> FileTransferError.
>
> Any objections?
>
> {
>       "code": FILE_NOT_FOUND_ERR
>       "source": "http://example.com/missing.txt",
>       "target": "file:///mnt/sdcard/foo.txt",
>       "http_status": 404
>       "http_message": "Not Found"
> }
>
> {
>       "code": CONNECTION_ERR
>       "source": "http://example.com/protected.txt",
>       "target": "file:///mnt/sdcard/bar.txt",
>       "http_status": 401
>       "http_message": "Authorization Required"
> }
>
> {
>       "code": CONNECTION_ERR
>       "source": "http://example.com/admin.txt",
>       "target": "file:///mnt/sdcard/baz.txt",
>       "http_status": 403
>       "http_message": "Forbidden"
> }
>

--047d7b2ed963cae74e04c00391bb--