Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Side by Side Diff: CHANGELOG.md

Issue 966063003: Overhaul the semantics of Request.handlerPath and Request.url. (Closed) Base URL: git@github.com:dart-lang/shelf@master
Patch Set: Code review changes Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | README.md » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 ## 0.6.0
2
3 **Breaking change:** The semantics of `Request.scriptName` and
4 [`Request.url`][url] have been overhauled, and the former has been renamed to
5 [`Request.handlerPath`][handlerPath]. `handlerPath` is now the root-relative URL
6 path to the current handler, while `url`'s path is the relative path from the
7 current handler to the requested. The new semantics are easier to describe and
8 to understand.
9
10 [url]: http://www.dartdocs.org/documentation/shelf/latest/index.html#shelf/shelf .Request@id_url
11 [handlerPath]: http://www.dartdocs.org/documentation/shelf/latest/index.html#she lf/shelf.Request@id_handlerPath
12
13 Practically speaking, the main difference is that the `/` at the beginning of
14 `url`'s path has been moved to the end of `handlerPath`. This makes `url`'s path
15 easier to parse using the `path` package.
16
17 [`Request.change`][change]'s handling of `handlerPath` and `url` has also
18 changed. Instead of taking both parameters separately and requiring that the
19 user manually maintain all the associated guarantees, it now takes a single
20 `path` parameter. This parameter is the relative path from the current
21 `handlerPath` to the next one, and sets both `handlerPath` and `url` on the new
22 `Request` accordingly.
23
24 [change]: http://www.dartdocs.org/documentation/shelf/latest/index.html#shelf/sh elf.Request@id_change
25
1 ## 0.5.7 26 ## 0.5.7
2 27
3 * Updated `Request` to support the `body` model from `Response`. 28 * Updated `Request` to support the `body` model from `Response`.
4 29
5 ## 0.5.6 30 ## 0.5.6
6 31
7 * Fixed `createMiddleware` to only catch errors if `errorHandler` is provided. 32 * Fixed `createMiddleware` to only catch errors if `errorHandler` is provided.
8 33
9 * Updated `handleRequest` in `shelf_io` to more gracefully handle errors when 34 * Updated `handleRequest` in `shelf_io` to more gracefully handle errors when
10 parsing `HttpRequest`. 35 parsing `HttpRequest`.
11 36
12 ## 0.5.5+1 37 ## 0.5.5+1
13 38
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 152
128 * **BREAKING** Removed `Shelf` prefix from all classes. 153 * **BREAKING** Removed `Shelf` prefix from all classes.
129 * **BREAKING** `Response` has drastically different constructors. 154 * **BREAKING** `Response` has drastically different constructors.
130 * *NEW!* `Response` now accepts a body of either `String` or 155 * *NEW!* `Response` now accepts a body of either `String` or
131 `Stream<List<int>>`. 156 `Stream<List<int>>`.
132 * *NEW!* `Response` now exposes `encoding` and `mimeType`. 157 * *NEW!* `Response` now exposes `encoding` and `mimeType`.
133 158
134 ## 0.1.0 2014-03-02 159 ## 0.1.0 2014-03-02
135 160
136 * First reviewed release 161 * First reviewed release
OLDNEW
« no previous file with comments | « no previous file | README.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698