Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add JSON validator sample #2815

Merged
merged 6 commits into from
Jan 24, 2024
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/clippy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ jobs:
cargo clippy -p sample_bits &&
cargo clippy -p sample_com_uri &&
cargo clippy -p sample_component_hello_world &&
cargo clippy -p sample_component_json_validator &&
cargo clippy -p sample_consent &&
cargo clippy -p sample_core_app &&
cargo clippy -p sample_counter &&
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ jobs:
cargo test -p sample_bits &&
cargo test -p sample_com_uri &&
cargo test -p sample_component_hello_world &&
cargo test -p sample_component_json_validator &&
cargo test -p sample_consent &&
cargo test -p sample_core_app &&
cargo test -p sample_counter &&
Expand Down Expand Up @@ -102,8 +103,8 @@ jobs:
cargo test -p test_dispatch &&
cargo test -p test_does_not_return &&
cargo test -p test_enums &&
cargo test -p test_error &&
cargo clean &&
cargo test -p test_error &&
cargo test -p test_event &&
cargo test -p test_extensions &&
cargo test -p test_handles &&
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/windows-sys.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
name: Check
strategy:
matrix:
rust: [1.56.0, stable, nightly]
rust: [1.60.0, stable, nightly]
runs-on:
- windows-latest
- ubuntu-latest
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/windows-version.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
name: Check
strategy:
matrix:
rust: [1.56.0, stable, nightly]
rust: [1.60.0, stable, nightly]
runs-on:
- windows-latest
- ubuntu-latest
Expand Down
2 changes: 1 addition & 1 deletion crates/libs/sys/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows-sys"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Rust for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/libs/targets/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ name = "windows-targets"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Import libs for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/libs/version/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows-version"
version = "0.1.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Windows version information"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
18 changes: 18 additions & 0 deletions crates/samples/components/json_validator/Cargo.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
[package]
name = "sample_component_json_validator"
version = "0.0.0"
edition = "2021"
publish = false

[lib]
crate-type = ["cdylib"]

[dependencies]
jsonschema = "0.17"
serde_json = "1.0"

[dependencies.windows]
path = "../../../libs/windows"
features = [
"Win32_Foundation",
]
1 change: 1 addition & 0 deletions crates/samples/components/json_validator/readme.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Sample for upcoming entry in [Getting Started with Rust](https://kennykerr.ca/rust-getting-started).
189 changes: 189 additions & 0 deletions crates/samples/components/json_validator/src/lib.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,189 @@
use jsonschema::JSONSchema;
use windows::{core::*, Win32::Foundation::*};

// Creates a JSON validator object with the given schema. The returned handle must be freed
// by calling `CloseJsonValidator`.
#[no_mangle]
unsafe extern "system" fn CreateJsonValidator(
schema: *const u8,
schema_len: usize,
handle: *mut usize,
) -> HRESULT {
create_validator(schema, schema_len, handle).into()
}

// Validates a JSON value against a previously-compiled schema.
#[no_mangle]
unsafe extern "system" fn ValidateJson(
handle: usize,
value: *const u8,
value_len: usize,
) -> HRESULT {
validate(handle, value, value_len).into()
}

// Closes a JSON validator object.
#[no_mangle]
unsafe extern "system" fn CloseJsonValidator(handle: usize) {
if handle != 0 {
_ = Box::from_raw(handle as *mut JSONSchema);
}
}

// Implementation of the `CreateJsonValidator` function so we can use `Result` for simplicity.
unsafe fn create_validator(schema: *const u8, schema_len: usize, handle: *mut usize) -> Result<()> {
let schema = json_from_raw_parts(schema, schema_len)?;

let compiled = JSONSchema::compile(&schema)
.map_err(|error| Error::new(E_INVALIDARG, error.to_string().into()))?;

if handle.is_null() {
return Err(E_POINTER.into());
}

// The handle is not null so we can safely dereference it here.
*handle = Box::into_raw(Box::new(compiled)) as usize;

Ok(())
}

// Implementation of the `ValidateJson` function so we can use `Result` for simplicity.
unsafe fn validate(handle: usize, value: *const u8, value_len: usize) -> Result<()> {
kennykerr marked this conversation as resolved.
Show resolved Hide resolved
if handle == 0 {
return Err(E_HANDLE.into());
}

let value = json_from_raw_parts(value, value_len)?;

// This looks a bit tricky but we're just turning the opaque handle into `JSONSchema` pointer
// and then returning a reference to avoid taking ownership of it.
let schema = &*(handle as *const JSONSchema);

if schema.is_valid(&value) {
Ok(())
} else {
let mut message = String::new();

// The `validate` method returns a collection of errors. We'll just return the first
// for simplicity.
if let Some(error) = schema.validate(&value).unwrap_err().next() {
message = error.to_string();
}

Err(Error::new(E_INVALIDARG, message.into()))
}
}

// Takes care of all the JSON parsing and parameter validation.
unsafe fn json_from_raw_parts(value: *const u8, value_len: usize) -> Result<serde_json::Value> {
if value.is_null() {
return Err(E_POINTER.into());
}

let value = std::slice::from_raw_parts(value, value_len);

let value =
std::str::from_utf8(value).map_err(|_| Error::from(ERROR_NO_UNICODE_TRANSLATION))?;

serde_json::from_str(value).map_err(|error| Error::new(E_INVALIDARG, format!("{error}").into()))
}

#[test]
fn simple() {
unsafe {
// Create a validator with the given schema.
let schema = r#"{"maxLength": 5}"#;
let mut handle = 0;
assert_eq!(
S_OK,
CreateJsonValidator(schema.as_ptr(), schema.len(), &mut handle)
);

// Validate the json against the schema.
let value = r#""Hello""#;
assert_eq!(S_OK, ValidateJson(handle, value.as_ptr(), value.len()));

// Check check validation failure provides reasonable error information.
let value = r#""Hello World""#;
let code = ValidateJson(handle, value.as_ptr(), value.len());
assert_eq!(E_INVALIDARG, code);
assert_eq!(
r#""Hello World" is longer than 5 characters"#,
Error::from(code).message()
);

// The schema validator is reusable.
let value = r#""World""#;
assert_eq!(S_OK, ValidateJson(handle, value.as_ptr(), value.len()));

// Close the validator with the given handle.
CloseJsonValidator(handle);

// Closing a "zero" handle is harmless.
CloseJsonValidator(0);
}
}

#[test]
fn invalid_create_params() {
unsafe {
// Check schema parsing failure provides reasonable error information.
let schema = r#"{ "invalid"#;
let mut handle = 0;
let code = CreateJsonValidator(schema.as_ptr(), schema.len(), &mut handle);
assert_eq!(E_INVALIDARG, code);
assert_eq!(
"EOF while parsing a string at line 1 column 10",
Error::from(code).message()
);

// Check that schema null pointer is caught.
let schema = r#"{"maxLength": 5}"#;
let mut handle = 0;
assert_eq!(
E_POINTER,
CreateJsonValidator(std::ptr::null(), schema.len(), &mut handle)
);

// Check that handle null pointer is caught.
assert_eq!(
E_POINTER,
CreateJsonValidator(schema.as_ptr(), schema.len(), std::ptr::null_mut())
);
}
}

#[test]
fn invalid_validate_params() {
unsafe {
// Create a validator with the given schema.
let schema = r#"{"maxLength": 5}"#;
let mut handle = 0;
assert_eq!(
S_OK,
CreateJsonValidator(schema.as_ptr(), schema.len(), &mut handle)
);

// Check that a zero handle is caught.
let value = r#""Hello""#;
assert_eq!(E_HANDLE, ValidateJson(0, value.as_ptr(), value.len()));

// Check that a value null pointer is caught.
assert_eq!(
E_POINTER,
ValidateJson(handle, std::ptr::null(), value.len())
);

// Check that JSON parsing failure provides reasonable error information.
let value = r#""Hello"#;
let code = ValidateJson(handle, value.as_ptr(), value.len());
assert_eq!(E_INVALIDARG, code);
assert_eq!(
"EOF while parsing a string at line 1 column 6",
Error::from(code).message()
);

// Close the validator with the given handle.
CloseJsonValidator(handle);
}
}
2 changes: 1 addition & 1 deletion crates/targets/aarch64_gnullvm/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows_aarch64_gnullvm"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Import lib for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/targets/aarch64_msvc/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows_aarch64_msvc"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Import lib for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/targets/i686_gnu/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows_i686_gnu"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Import lib for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/targets/i686_msvc/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows_i686_msvc"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Import lib for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/targets/x86_64_gnu/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows_x86_64_gnu"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Import lib for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/targets/x86_64_gnullvm/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows_x86_64_gnullvm"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Import lib for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/targets/x86_64_msvc/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "windows_x86_64_msvc"
version = "0.52.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Import lib for Windows"
repository = "https://github.com/microsoft/windows-rs"
Expand Down
2 changes: 1 addition & 1 deletion crates/tools/riddle/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name = "riddle"
version = "0.1.0"
authors = ["Microsoft"]
edition = "2021"
rust-version = "1.56"
rust-version = "1.60"
license = "MIT OR Apache-2.0"
description = "Windows metadata compiler"
repository = "https://github.com/microsoft/windows-rs"
Expand Down