/*
 * Copyright © 2015 Cask Data, Inc.
 *
 * Licensed under the Apache License, Version 2.0 (the "License"); you may not
 * use this file except in compliance with the License. You may obtain a copy of
 * the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 * License for the specific language governing permissions and limitations under
 * the License.
 */

package co.cask.cdap.common.exception;

import co.cask.cdap.common.http.SecurityRequestContext;
import co.cask.http.ExceptionHandler;
import co.cask.http.HttpResponder;
import org.jboss.netty.handler.codec.http.HttpRequest;
import org.jboss.netty.handler.codec.http.HttpResponseStatus;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

/**
 * Common logic to handle exceptions in handler methods.
 */
public class HttpExceptionHandler extends ExceptionHandler {

  private static final Logger LOG = LoggerFactory.getLogger(HttpExceptionHandler.class);

  @Override
  public void handle(Throwable t, HttpRequest request, HttpResponder responder) {
    if (t instanceof IllegalArgumentException) {
      // TODO: use BadRequestException instead
      responder.sendString(HttpResponseStatus.BAD_REQUEST, t.getMessage());
    } else if (t instanceof BadRequestException) {
      responder.sendString(HttpResponseStatus.BAD_REQUEST, t.getMessage());
    } else if (t instanceof ConflictException) {
      responder.sendString(HttpResponseStatus.CONFLICT, t.getMessage());
    } else if (t instanceof NotFoundException) {
      responder.sendString(HttpResponseStatus.NOT_FOUND, t.getMessage());
    } else if (t instanceof UnsupportedOperationException) {
      responder.sendStatus(HttpResponseStatus.NOT_IMPLEMENTED);
    } else if (t instanceof UnauthorizedException) {
      responder.sendStatus(HttpResponseStatus.UNAUTHORIZED);
    } else {
      LOG.error("Unexpected error: request={} {} user={}:",
                request.getMethod().getName(), request.getUri(),
                SecurityRequestContext.getUserId().or("<null>"), t);
      responder.sendStatus(HttpResponseStatus.INTERNAL_SERVER_ERROR);
    }
  }

}

@@ -1271,187 +1271,8 @@
 eof 
-IllegalArgumentException) %7B%0A      // TODO: use BadRequestException instead%0A      responder.sendString(HttpResponseStatus.BAD_REQUEST, t.getMessage());%0A    %7D else if (t instanceof 
 BadR
@@ -1615,133 +1615,8 @@
 ));%0A
-    %7D else if (t instanceof UnsupportedOperationException) %7B%0A      responder.sendStatus(HttpResponseStatus.NOT_IMPLEMENTED);%0A
     

Remove handling of IllegalArgumentException and UnsupportedOperationException in HttpExceptionHandler