/*-
 * -\-\-
 * FastForward HTTP Module
 * --
 * Copyright (C) 2016 - 2018 Spotify AB
 * --
 * 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 com.spotify.ffwd.http;

import static io.netty.handler.codec.http.HttpMethod.GET;
import static io.netty.handler.codec.http.HttpMethod.POST;

import com.fasterxml.jackson.databind.ObjectMapper;
import com.spotify.ffwd.model.Batch;
import io.netty.buffer.ByteBufInputStream;
import io.netty.channel.ChannelFutureListener;
import io.netty.channel.ChannelHandler.Sharable;
import io.netty.channel.ChannelHandlerContext;
import io.netty.handler.codec.MessageToMessageDecoder;
import io.netty.handler.codec.http.DefaultFullHttpResponse;
import io.netty.handler.codec.http.FullHttpRequest;
import io.netty.handler.codec.http.HttpResponseStatus;
import io.netty.handler.codec.http.HttpVersion;
import java.io.IOException;
import java.io.InputStream;
import java.util.List;
import javax.inject.Inject;
import javax.inject.Named;
import lombok.extern.slf4j.Slf4j;

@Slf4j
@Sharable
public class HttpDecoder extends MessageToMessageDecoder<FullHttpRequest> {
    @Inject
    @Named("application/json")
    private ObjectMapper mapper;

    @Override
    protected void decode(ChannelHandlerContext ctx, FullHttpRequest in, List<Object> out)
        throws Exception {
        switch (in.uri()) {
            case "/ping":
                if (in.method() == GET) {
                    getPing(ctx, in, out);
                    return;
                }

                throw new HttpException(HttpResponseStatus.METHOD_NOT_ALLOWED);
            case "/v1/batch":
                if (in.method() == POST) {
                    if (matchContentType(in, "application/json")) {
                        postBatch(ctx, in, out);
                        return;
                    }
                    log.error("Unsupported Media Type");
                    throw new HttpException(HttpResponseStatus.UNSUPPORTED_MEDIA_TYPE);
                }
                log.error("HTTP Method Not Allowed");
                throw new HttpException(HttpResponseStatus.METHOD_NOT_ALLOWED);
            default:
                /* do nothing */
                break;
        }

        throw new HttpException(HttpResponseStatus.NOT_FOUND);
    }

    private boolean matchContentType(final FullHttpRequest in, final String expected) {
        final String value = in.headers().get("Content-Type");
        return value != null && value.equals(expected);
    }

    private void postBatch(
        final ChannelHandlerContext ctx, final FullHttpRequest in, final List<Object> out
    ) {
        final Batch batch;
        try (final InputStream inputStream = new ByteBufInputStream(in.content())) {
            batch = mapper.readValue(inputStream, Batch.class);
        } catch (final IOException e) {
            log.error("HTTP Bad Request", e);
            throw new HttpException(HttpResponseStatus.BAD_REQUEST);
        }

        out.add(batch);

        ctx
            .channel()
            .writeAndFlush(new DefaultFullHttpResponse(HttpVersion.HTTP_1_1, HttpResponseStatus.OK))
            .addListener((ChannelFutureListener) future -> future.channel());
    }

    private void getPing(
        final ChannelHandlerContext ctx, final FullHttpRequest in, final List<Object> out
    ) {
        ctx
            .channel()
            .writeAndFlush(new DefaultFullHttpResponse(HttpVersion.HTTP_1_1, HttpResponseStatus.OK))
            .addListener((ChannelFutureListener) future -> future.channel());
    }
}

@@ -3698,32 +3698,40 @@
 future.channel()
+.close()
 );%0A    %7D%0A%0A    pr
@@ -4059,16 +4059,24 @@
 hannel()
+.close()
 );%0A    %7D

revert change that removed `.close()`. sending metrics via curl/etc (#121)