GitHub PullRequestReviewCommentEvent at 2014-03-10 13:11:36
Repository: spray
Actor: sirthias
Payload: {"comment":{"url":"https://api.github.com/repos/2beaucoup/spray/pulls/comments/10428057","id":10428057,"diff_hunk":"@@ -34,7 +34,12 @@ trait RangeDirectives {\n    * This directive is transparent to non-GET requests\n    * Rejects requests with unsatisfiable ranges `UnsatisfiableRangeRejection`.\n    * Rejects requests with too many expected ranges.\n-   * @see rfc2616 14.35.2\n+   *\n+   * Note: if you want to combine this directive with `conditional(...)` you need to put\n+   * it on the *inside* of the `conditional(...)` directive, i.e. `conditional(...)` must be\n+   * on a higher level in your route structure in order to function correctly.\n+   *\n+   * @see https://datatracker.ietf.org/doc/draft-ietf-httpbis-p5-range/","path":"spray-routing/src/main/scala/spray/routing/directives/RangeDirectives.scala","position":10,"original_position":10,"commit_id":"9fb6e4d73270bfa32ae82b24ab6af48e61b04521","original_commit_id":"9fb6e4d73270bfa32ae82b24ab6af48e61b04521","user":{"login":"sirthias","id":135347,"avatar_url":"https://gravatar.com/avatar/ac1b5d9d839023af8f7833b75cd14b44?d=https%3A%2F%2Fidenticons.github.com%2Fcc61bf4b3ccd8b2cff7f6e5a657563f1.png&r=x","gravatar_id":"ac1b5d9d839023af8f7833b75cd14b44","url":"https://api.github.com/users/sirthias","html_url":"https://github.com/sirthias","followers_url":"https://api.github.com/users/sirthias/followers","following_url":"https://api.github.com/users/sirthias/following{/other_user}","gists_url":"https://api.github.com/users/sirthias/gists{/gist_id}","starred_url":"https://api.github.com/users/sirthias/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sirthias/subscriptions","organizations_url":"https://api.github.com/users/sirthias/orgs","repos_url":"https://api.github.com/users/sirthias/repos","events_url":"https://api.github.com/users/sirthias/events{/privacy}","received_events_url":"https://api.github.com/users/sirthias/received_events","type":"User","site_admin":false},"body":"True. `tools.ietf.org` is better.","created_at":"2014-03-10T13:11:36Z","updated_at":"2014-03-10T13:11:36Z","html_url":"https://github.com/2beaucoup/spray/pull/1#discussion_r10428057","pull_request_url":"https://api.github.com/repos/2beaucoup/spray/pulls/1","_links":{"self":{"href":"https://api.github.com/repos/2beaucoup/spray/pulls/comments/10428057"},"html":{"href":"https://github.com/2beaucoup/spray/pull/1#discussion_r10428057"},"pull_request":{"href":"https://api.github.com/repos/2beaucoup/spray/pulls/1"}}}}