2014-07-01 22:25:54 +02:00
|
|
|
package flatmap
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
Fix string representation of sets during interpolation
The change in #10787 used flatmap.Expand to fix interpolation of nested
maps, but it broke interpolation of sets such that their elements were
not represented. For example, the expected string representation of a
splatted aws_network_interface.whatever.*.private_ips should be:
```
[{Variable (TypeList): [{Variable (TypeString): 10.41.17.25}]} {Variable (TypeList): [{Variable (TypeString): 10.41.22.236}]}]
```
But instead it became:
```
[{Variable (TypeList): [{Variable (TypeString): }]} {Variable (TypeList): [{Variable (TypeString): }]}]
```
This is because the expandArray function of expand.go treated arrays to
exclusively be lists, e.g. not sets. The old code used to match for
numeric keys, so it would work for sets, whereas expandArray just
assumed keys started at 0 and ascended incrementally. Remember that
sets' keys are numeric, but since they are hashes, they can be any
integer. The result of assuming that the keys start at 0 led to the
recursive call to flatmap.Expand not matching any keys of the set, and
returning nil, which is why the above example has nothing where the IP
addresses used to be.
So we bring back that matching behavior, but we move it to expandArray
instead. We've modified it to not reconstruct the data structures like
it used to when it was in the Interpolator, and to use the standard int
sorter rather than implementing a custom sorter since a custom one is no
longer necessary thanks to the use of flatmap.Expand.
Fixes #10908, and restores the viability of the workaround I posted in #8696.
Big thanks to @jszwedko for helping me with this fix. I was able to
diagnose the problem along, but couldn't fix it without his help.
2016-12-23 23:59:18 +01:00
|
|
|
"sort"
|
2014-07-01 22:25:54 +02:00
|
|
|
"strconv"
|
|
|
|
"strings"
|
|
|
|
)
|
|
|
|
|
|
|
|
// Expand takes a map and a key (prefix) and expands that value into
|
|
|
|
// a more complex structure. This is the reverse of the Flatten operation.
|
|
|
|
func Expand(m map[string]string, key string) interface{} {
|
|
|
|
// If the key is exactly a key in the map, just return it
|
|
|
|
if v, ok := m[key]; ok {
|
2014-07-24 20:40:46 +02:00
|
|
|
if v == "true" {
|
2014-07-01 22:25:54 +02:00
|
|
|
return true
|
|
|
|
} else if v == "false" {
|
|
|
|
return false
|
|
|
|
}
|
|
|
|
|
|
|
|
return v
|
|
|
|
}
|
|
|
|
|
|
|
|
// Check if the key is an array, and if so, expand the array
|
|
|
|
if _, ok := m[key+".#"]; ok {
|
|
|
|
return expandArray(m, key)
|
|
|
|
}
|
|
|
|
|
|
|
|
// Check if this is a prefix in the map
|
|
|
|
prefix := key + "."
|
|
|
|
for k, _ := range m {
|
|
|
|
if strings.HasPrefix(k, prefix) {
|
|
|
|
return expandMap(m, prefix)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func expandArray(m map[string]string, prefix string) []interface{} {
|
|
|
|
num, err := strconv.ParseInt(m[prefix+".#"], 0, 0)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
|
|
|
}
|
|
|
|
|
2017-01-04 22:03:24 +01:00
|
|
|
keySet := map[int]bool{}
|
|
|
|
for k := range m {
|
|
|
|
if !strings.HasPrefix(k, prefix+".") {
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
key := k[len(prefix)+1:]
|
|
|
|
idx := strings.Index(key, ".")
|
|
|
|
if idx != -1 {
|
|
|
|
key = key[:idx]
|
|
|
|
}
|
|
|
|
|
|
|
|
// skip the count value
|
|
|
|
if key == "#" {
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
k, err := strconv.Atoi(key)
|
|
|
|
if err != nil {
|
|
|
|
panic(err)
|
Fix string representation of sets during interpolation
The change in #10787 used flatmap.Expand to fix interpolation of nested
maps, but it broke interpolation of sets such that their elements were
not represented. For example, the expected string representation of a
splatted aws_network_interface.whatever.*.private_ips should be:
```
[{Variable (TypeList): [{Variable (TypeString): 10.41.17.25}]} {Variable (TypeList): [{Variable (TypeString): 10.41.22.236}]}]
```
But instead it became:
```
[{Variable (TypeList): [{Variable (TypeString): }]} {Variable (TypeList): [{Variable (TypeString): }]}]
```
This is because the expandArray function of expand.go treated arrays to
exclusively be lists, e.g. not sets. The old code used to match for
numeric keys, so it would work for sets, whereas expandArray just
assumed keys started at 0 and ascended incrementally. Remember that
sets' keys are numeric, but since they are hashes, they can be any
integer. The result of assuming that the keys start at 0 led to the
recursive call to flatmap.Expand not matching any keys of the set, and
returning nil, which is why the above example has nothing where the IP
addresses used to be.
So we bring back that matching behavior, but we move it to expandArray
instead. We've modified it to not reconstruct the data structures like
it used to when it was in the Interpolator, and to use the standard int
sorter rather than implementing a custom sorter since a custom one is no
longer necessary thanks to the use of flatmap.Expand.
Fixes #10908, and restores the viability of the workaround I posted in #8696.
Big thanks to @jszwedko for helping me with this fix. I was able to
diagnose the problem along, but couldn't fix it without his help.
2016-12-23 23:59:18 +01:00
|
|
|
}
|
2017-01-04 22:03:24 +01:00
|
|
|
keySet[int(k)] = true
|
Fix string representation of sets during interpolation
The change in #10787 used flatmap.Expand to fix interpolation of nested
maps, but it broke interpolation of sets such that their elements were
not represented. For example, the expected string representation of a
splatted aws_network_interface.whatever.*.private_ips should be:
```
[{Variable (TypeList): [{Variable (TypeString): 10.41.17.25}]} {Variable (TypeList): [{Variable (TypeString): 10.41.22.236}]}]
```
But instead it became:
```
[{Variable (TypeList): [{Variable (TypeString): }]} {Variable (TypeList): [{Variable (TypeString): }]}]
```
This is because the expandArray function of expand.go treated arrays to
exclusively be lists, e.g. not sets. The old code used to match for
numeric keys, so it would work for sets, whereas expandArray just
assumed keys started at 0 and ascended incrementally. Remember that
sets' keys are numeric, but since they are hashes, they can be any
integer. The result of assuming that the keys start at 0 led to the
recursive call to flatmap.Expand not matching any keys of the set, and
returning nil, which is why the above example has nothing where the IP
addresses used to be.
So we bring back that matching behavior, but we move it to expandArray
instead. We've modified it to not reconstruct the data structures like
it used to when it was in the Interpolator, and to use the standard int
sorter rather than implementing a custom sorter since a custom one is no
longer necessary thanks to the use of flatmap.Expand.
Fixes #10908, and restores the viability of the workaround I posted in #8696.
Big thanks to @jszwedko for helping me with this fix. I was able to
diagnose the problem along, but couldn't fix it without his help.
2016-12-23 23:59:18 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
keysList := make([]int, 0, num)
|
|
|
|
for key := range keySet {
|
|
|
|
keysList = append(keysList, key)
|
|
|
|
}
|
|
|
|
sort.Ints(keysList)
|
|
|
|
|
2014-07-01 22:25:54 +02:00
|
|
|
result := make([]interface{}, num)
|
Fix string representation of sets during interpolation
The change in #10787 used flatmap.Expand to fix interpolation of nested
maps, but it broke interpolation of sets such that their elements were
not represented. For example, the expected string representation of a
splatted aws_network_interface.whatever.*.private_ips should be:
```
[{Variable (TypeList): [{Variable (TypeString): 10.41.17.25}]} {Variable (TypeList): [{Variable (TypeString): 10.41.22.236}]}]
```
But instead it became:
```
[{Variable (TypeList): [{Variable (TypeString): }]} {Variable (TypeList): [{Variable (TypeString): }]}]
```
This is because the expandArray function of expand.go treated arrays to
exclusively be lists, e.g. not sets. The old code used to match for
numeric keys, so it would work for sets, whereas expandArray just
assumed keys started at 0 and ascended incrementally. Remember that
sets' keys are numeric, but since they are hashes, they can be any
integer. The result of assuming that the keys start at 0 led to the
recursive call to flatmap.Expand not matching any keys of the set, and
returning nil, which is why the above example has nothing where the IP
addresses used to be.
So we bring back that matching behavior, but we move it to expandArray
instead. We've modified it to not reconstruct the data structures like
it used to when it was in the Interpolator, and to use the standard int
sorter rather than implementing a custom sorter since a custom one is no
longer necessary thanks to the use of flatmap.Expand.
Fixes #10908, and restores the viability of the workaround I posted in #8696.
Big thanks to @jszwedko for helping me with this fix. I was able to
diagnose the problem along, but couldn't fix it without his help.
2016-12-23 23:59:18 +01:00
|
|
|
for i, key := range keysList {
|
|
|
|
result[i] = Expand(m, fmt.Sprintf("%s.%d", prefix, key))
|
2014-07-01 22:25:54 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
return result
|
|
|
|
}
|
|
|
|
|
|
|
|
func expandMap(m map[string]string, prefix string) map[string]interface{} {
|
|
|
|
result := make(map[string]interface{})
|
|
|
|
for k, _ := range m {
|
|
|
|
if !strings.HasPrefix(k, prefix) {
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
key := k[len(prefix):]
|
|
|
|
idx := strings.Index(key, ".")
|
2014-07-08 22:57:30 +02:00
|
|
|
if idx != -1 {
|
|
|
|
key = key[:idx]
|
|
|
|
}
|
|
|
|
if _, ok := result[key]; ok {
|
|
|
|
continue
|
2014-07-01 22:25:54 +02:00
|
|
|
}
|
|
|
|
|
2016-12-16 15:35:58 +01:00
|
|
|
// skip the map count value
|
|
|
|
if key == "%" {
|
|
|
|
continue
|
|
|
|
}
|
2014-07-08 22:57:30 +02:00
|
|
|
result[key] = Expand(m, k[:len(prefix)+len(key)])
|
2014-07-01 22:25:54 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
return result
|
|
|
|
}
|