Concatenated strings as keys

[mslinn]

Given the following:

repositories {
    AWS S3 : "http://jets3t.s3.amazonaws.com/maven2"
}

The key is parsed as ""AWS S3"" instead of "AWS S3", which is what I expected from reading the README.

[havocp]

I added the following test and it passed. My guess is that you're expecting to use a string with methods that expect a path expression, see the docs for class Config in http://typesafehub.github.com/config/latest/api/

If that isn't it then it would be helpful to modify the test below to break.

    @Test
    def unquotedKeyWithWhitespace() {
        val conf = ConfigFactory.parseString("""repositories {
    AWS S3 : "http://jets3t.s3.amazonaws.com/maven2"
}""")
        assertEquals("http://jets3t.s3.amazonaws.com/maven2", conf.getObject("repositories").get("AWS S3").unwrapped.asInstanceOf[String])
        assertEquals("http://jets3t.s3.amazonaws.com/maven2", conf.getString("repositories.\"AWS S3\""))
    }

[mslinn]

I've got code like this (you can see the whole file here)

val entireConfig: Config = ConfigFactory.parseURL(new URL(fetchFromUrl))

  /** Work around Config bug */
  def sanitizeKey(key: String) = key.replace("\"", " ").trim

  def makeSettings(lookup: Lookup): SortedMap[String, String] = {
    lookup.config.entrySet foreach { kv =>
      val key = kv.getKey
      val value = kv.getValue.unwrapped.toString
      keyValues.put(sanitizeKey(key), value)
    }
    SortedMap.empty[String, String] ++ (keyValues.toList.sortBy(_._1))
  }

  def apply(key: String) = {
    val sanitizedKey = sanitizeKey(key)
    val value = keyValues.get(sanitizedKey)
    if (value!=None) {
      if (!quiet && !alreadyShown.contains(sanitizedKey)) {
        alreadyShown += sanitizedKey
        println("  %s = %s".format(sanitizedKey, value.get))
      }
      value.get
    } else {
      println("Warning: %s is not defined in the config file at %s".format(sanitizedKey, fetchFromUrl))
      ""
    }
  }

[havocp]

I think your problem is that entrySet on a Config has path expressions as keys. Try config.root.entrySet (but of course you then have a tree of maps, not a single level).

[mslinn]

I'll stick with my workaround. Might be worthwhile mentioning something about this corner case in the docs.

[havocp]

I think it is mentioned but I'll see if
it can go in more places. To convert between paths and keys, you need to do more than your sanitizeKey does; there are utility methods joinPath and splitPath in ConfigUtil for this. or something like that, I may remember the wrong method names.