Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(42)

Issue 1708051: Use default container config for JsServlet

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 8 months ago by zhoresh
Modified:
15 years, 8 months ago
Reviewers:
johnfargo, shindig-dev
Base URL:
http://svn.apache.org/repos/asf/shindig/trunk
Visibility:
Public.

Description

Also reply with 404 instead of 500 for bad Js request

Patch Set 1 #

Patch Set 2 : Sync with head #

Unified diffs Side-by-side diffs Delta from patch set Stats (+126 lines, -75 lines) Patch
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/JsServlet.java View 1 7 chunks +13 lines, -6 lines 0 comments Download
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultJsUriManager.java View 1 7 chunks +32 lines, -26 lines 0 comments Download
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/JsUriManager.java View 1 3 chunks +10 lines, -9 lines 0 comments Download
java/gadgets/src/test/java/org/apache/shindig/gadgets/uri/DefaultJsUriManagerTest.java View 1 17 chunks +71 lines, -34 lines 0 comments Download

Messages

Total messages: 4
zhoresh
15 years, 8 months ago (2010-07-02 20:57:51 UTC) #1
zhoresh
Sync with head
15 years, 8 months ago (2010-07-02 21:06:35 UTC) #2
johnfargo
LGTM On 2010/07/02 21:06:35, zhoresh wrote: > Sync with head
15 years, 8 months ago (2010-07-02 21:09:34 UTC) #3
zhoresh
15 years, 8 months ago (2010-07-02 22:00:50 UTC) #4
Submitted
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b