Fix #182 Set section name for workspace/configuration

Signed-off-by: Remy Suen <[email protected]>
rcjsuen · Oct 23, 2017 · c0d0d7a · c0d0d7a
1 parent 7d1599b
commit c0d0d7a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@ All notable changes to this project will be documented in this file.
   - create docker.command.flagToChown to convert an unknown ADD or COPY flag to a --chown ([#187](https://github.com/rcjsuen/dockerfile-language-server-nodejs/issues/187))
 ### Fixed
 - use a reasonable range for the diagnostic if an unknown flag has no name ([#186](https://github.com/rcjsuen/dockerfile-language-server-nodejs/issues/186))
+- specify a section name when sending a workspace/configuration request ([#182](https://github.com/rcjsuen/dockerfile-language-server-nodejs/issues/182))
 
 ## [0.0.9] - 2017-10-14
 ### Added

diff --git a/src/server.ts b/src/server.ts
@@ -217,7 +217,7 @@ function getSeverity(severity: string | undefined): ValidationSeverity | null {
 function getConfiguration(resource: string): Thenable<ValidatorConfiguration> {
 	let result = validatorConfigurations.get(resource);
 	if (!result) {
-		result = connection.workspace.getConfiguration({ section: '', scopeUri: resource });
+		result = connection.workspace.getConfiguration({ section: "docker.languageserver.diagnostics", scopeUri: resource });
 		validatorConfigurations.set(resource, result);
 	}
 	return result;
@@ -235,7 +235,7 @@ function refreshConfigurations() {
 	// store all the URIs that need to be refreshed
 	const settingsRequest: ConfigurationItem[] = [];
 	for (let uri in documents) {
-		settingsRequest.push({ section: "", scopeUri: uri });
+		settingsRequest.push({ section: "docker.languageserver.diagnostics", scopeUri: uri });
 	}
 	// clear the cache
 	validatorConfigurations.clear();