From d4125f217b1a1a388c0f0abc23ec6f548ea0c06e Mon Sep 17 00:00:00 2001
From: Havoc Pennington <hp@pobox.com>
Date: Mon, 2 Mar 2015 16:54:50 -0500
Subject: [PATCH] Set Accept header on URL connections

This fixes #182 in theory, though test coverage is nonexistent.
---
 .../com/typesafe/config/impl/Parseable.java   | 35 ++++++++++++++++++-
 1 file changed, 34 insertions(+), 1 deletion(-)

diff --git a/config/src/main/java/com/typesafe/config/impl/Parseable.java b/config/src/main/java/com/typesafe/config/impl/Parseable.java
index 5f301aa1..616ffaa1 100644
--- a/config/src/main/java/com/typesafe/config/impl/Parseable.java
+++ b/config/src/main/java/com/typesafe/config/impl/Parseable.java
@@ -100,6 +100,10 @@ public abstract class Parseable implements ConfigParseable {
     // to support the "allow missing" feature.
     protected abstract Reader reader() throws IOException;
 
+    protected Reader reader(ConfigParseOptions options) throws IOException {
+        return reader();
+    }
+
     protected static void trace(String message) {
         if (ConfigImpl.traceLoadsEnabled()) {
             ConfigImpl.trace(message);
@@ -195,7 +199,7 @@ public abstract class Parseable implements ConfigParseable {
     // options.getAllowMissing()
     protected AbstractConfigValue rawParseValue(ConfigOrigin origin, ConfigParseOptions finalOptions)
             throws IOException {
-        Reader reader = reader();
+        Reader reader = reader(finalOptions);
 
         // after reader() we will have loaded the Content-Type.
         ConfigSyntax contentType = contentType();
@@ -437,9 +441,38 @@ public abstract class Parseable implements ConfigParseable {
 
         @Override
         protected Reader reader() throws IOException {
+            throw new ConfigException.BugOrBroken("reader() without options should not be called on ParseableURL");
+        }
+
+        private static String acceptContentType(ConfigParseOptions options) {
+            if (options.getSyntax() == null)
+                return null;
+
+            switch (options.getSyntax()) {
+            case JSON:
+                return jsonContentType;
+            case CONF:
+                return hoconContentType;
+            case PROPERTIES:
+                return propertiesContentType;
+            }
+
+            // not sure this is reachable but javac thinks it is
+            return null;
+        }
+
+        @Override
+        protected Reader reader(ConfigParseOptions options) throws IOException {
             if (ConfigImpl.traceLoadsEnabled())
                 trace("Loading config from a URL: " + input.toExternalForm());
             URLConnection connection = input.openConnection();
+
+            // allow server to serve multiple types from one URL
+            String acceptContent = acceptContentType(options);
+            if (acceptContent != null) {
+                connection.setRequestProperty("Accept", acceptContent);
+            }
+
             connection.connect();
 
             // save content type for later