@@ -204,13 +204,19 @@ private async Task<string> GetValueAsync(string key)
204204 if ( _client == null )
205205 return null ;
206206
207- SettingSelector selector = new SettingSelector ( key , _labelFilter ) ;
208- // TODO: Reduce bandwidth by limiting the fields we retrieve.
209- //selector.Fields = SettingFields.Key | SettingFields.Value | SettingFields.ContentType;
207+ SettingSelector selector = new SettingSelector ( ) ;
208+ selector . KeyFilter = key ;
209+ if ( _labelFilter != null )
210+ {
211+ selector . LabelFilter = _labelFilter ;
212+ }
210213 if ( _dateTimeFilter > DateTimeOffset . MinValue )
211214 {
212215 selector . AcceptDateTime = _dateTimeFilter ;
213216 }
217+ // TODO: Reduce bandwidth by limiting the fields we retrieve.
218+ // Currently, content type doesn't get delivered, even if we add it to the selection. This prevents KeyVault recognition.
219+ //selector.Fields = SettingFields.Key | SettingFields.Value | SettingFields.ContentType;
214220
215221 try
216222 {
@@ -260,13 +266,22 @@ private async Task<ICollection<KeyValuePair<string, string>>> GetAllValuesAsync(
260266 if ( _client == null )
261267 return data ;
262268
263- SettingSelector selector = new SettingSelector ( _keyFilter , _labelFilter ) ;
264- // TODO: Reduce bandwidth by limiting the fields we retrieve.
265- //selector.Fields = SettingFields.Key | SettingFields.Value | SettingFields.ContentType;
269+ SettingSelector selector = new SettingSelector ( ) ;
270+ if ( _keyFilter != null )
271+ {
272+ selector . KeyFilter = _keyFilter ;
273+ }
274+ if ( _labelFilter != null )
275+ {
276+ selector . LabelFilter = _labelFilter ;
277+ }
266278 if ( _dateTimeFilter > DateTimeOffset . MinValue )
267279 {
268280 selector . AcceptDateTime = _dateTimeFilter ;
269281 }
282+ // TODO: Reduce bandwidth by limiting the fields we retrieve.
283+ // Currently, content type doesn't get delivered, even if we add it to the selection. This prevents KeyVault recognition.
284+ //selector.Fields = SettingFields.Key | SettingFields.Value | SettingFields.ContentType;
270285
271286 // We don't make any guarantees about which kv get precendence when there are multiple of the same key...
272287 // But the config service does seem to return kvs in a preferred order - no label first, then alphabetical by label.
0 commit comments