Skip to content

Commit b0e6ed9

Browse files
authored
Merge pull request #457 from SteveL-MSFT/remove-schema-url
remove support to retrieve schema from URL
2 parents f722880 + dd99038 commit b0e6ed9

File tree

4 files changed

+0
-22
lines changed

4 files changed

+0
-22
lines changed

dsc_lib/Cargo.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ indicatif = { version = "0.17" }
1111
jsonschema = "0.17"
1212
num-traits = "0.2"
1313
regex = "1.7"
14-
reqwest = { version = "0.12", features = ["blocking"] }
1514
schemars = { version = "0.8.12", features = ["preserve_order"] }
1615
serde = { version = "1.0", features = ["derive"] }
1716
serde_json = { version = "1.0", features = ["preserve_order"] }

dsc_lib/src/dscerror.rs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
use std::str::Utf8Error;
55

66
use indicatif::style::TemplateError;
7-
use reqwest::StatusCode;
87
use thiserror::Error;
98
use tracing::error;
109
use tree_sitter::LanguageError;
@@ -32,12 +31,6 @@ pub enum DscError {
3231
#[error("Function '{0}' error: {1}")]
3332
FunctionArg(String, String),
3433

35-
#[error("HTTP: {0}")]
36-
Http(#[from] reqwest::Error),
37-
38-
#[error("HTTP status: {0}")]
39-
HttpStatus(StatusCode),
40-
4134
#[error("Function integer argument conversion error: {0}")]
4235
IntegerConversion(#[from] std::num::ParseIntError),
4336

dsc_lib/src/dscresources/command_resource.rs

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -465,17 +465,6 @@ pub fn get_schema(resource: &ResourceManifest, cwd: &str) -> Result<String, DscE
465465
let json = serde_json::to_string(schema)?;
466466
Ok(json)
467467
},
468-
SchemaKind::Url(ref url) => {
469-
// TODO: cache downloaded schemas so we don't have to download them every time
470-
let mut response = reqwest::blocking::get(url)?;
471-
if !response.status().is_success() {
472-
return Err(DscError::HttpStatus(response.status()));
473-
}
474-
475-
let mut body = String::new();
476-
response.read_to_string(&mut body)?;
477-
Ok(body)
478-
},
479468
}
480469
}
481470

dsc_lib/src/dscresources/resource_manifest.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,9 +126,6 @@ pub enum SchemaKind {
126126
/// The schema is embedded in the manifest.
127127
#[serde(rename = "embedded")]
128128
Embedded(Value),
129-
/// The schema is retrieved from a URL. Required for intellisense support.
130-
#[serde(rename = "url")]
131-
Url(String),
132129
}
133130

134131
#[derive(Debug, Clone, PartialEq, Deserialize, Serialize, JsonSchema)]

0 commit comments

Comments
 (0)