8000 fix(deps): update rust crate wasm-bindgen-futures to 0.4.42 by renovate[bot] · Pull Request #490 · the-guild-org/conductor · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Mar 18, 2025. It is now read-only.

fix(deps): update rust crate wasm-bindgen-futures to 0.4.42 #490

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

renovate[bot]
Copy link
Contributor
@renovate renovate bot commented Mar 4, 2024

Mend Renovate

This PR contains the following updates:

Package Type Update Change
wasm-bindgen-futures (source) dependencies patch 0.4.41 -> 0.4.42

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

Copy link
github-actions bot commented Mar 4, 2024

🚨 Rust Panic Audit: 102 Potential Panic Points Detected 🚨

Crate: federation_query_planner

📊 Total Usages: 53

  • 🎁 unwrap usages: 32
  • 🚨 panic usages: 3
  • 🔎 expect usages: 8
  • 🔢 array_index usages: 10

Crate: common

📊 Total Usages: 11

  • 🔢 array_index usages: 1
  • 🎁 unwrap usages: 10

Crate: telemetry

📊 Total Usages: 9

  • 🔢 array_index usages: 5
  • 🎁 unwrap usages: 4

Crate: cloudflare_worker

📊 Total Usages: 8

  • 🚨 panic usages: 1
  • 🔎 expect usages: 2
  • 🎁 unwrap usages: 5

Crate: engine

📊 Total Usages: 7

  • 🔎 expect usages: 1
  • 🎁 unwrap usages: 6

Crate: tracing

📊 Total Usages: 6

  • 🎁 unwrap usages: 5
  • 🔎 expect usages: 1

Crate: conductor

📊 Total Usages: 5

  • 🚨 panic usages: 1
  • 🔎 expect usages: 3
  • 🎁 unwrap usages: 1

Crate: config

📊 Total Usages: 3

  • 🎁 unwrap usages: 2
  • 🚨 panic usages: 1

📌 Expected Annotations

Crate: vrl 8000

📊 Total Expected Usages: 2

expand details
  1. Reason: "if the provided VRL code in the config file can't compile, we have to exit."
  • Code: panic!("failed to compile vrl program");
  • Location: ./plugins/vrl/src/plugin.rs:129
  1. Reason: "states is a non-user provided variable"
  • Code: .expect("can't merge states when states is an empty vector!")
  • Location: ./plugins/vrl/src/plugin.rs:146

Crate: conductor

📊 Total Expected Usages: 2

expand details
  1. Reason: "we need to exit the process, if the logger can't be correctly set."
  • Code: let _guard = tracing::subscriber::set_default(subscriber);
  • Location: ./bin/conductor/src/lib.rs:37
  1. Reason: "we need to exit the process, if the provided configuration file is incorrect."
  • Code: panic!("Failed to initialize gateway: {:?}", e);
  • Location: ./bin/conductor/src/lib.rs:76

Crate: config

📊 Total Expected Usages: 9

expand details
  1. Reason: "part of development docgen CLI"
  • Code: .expect("Failed to serialize json schema for config file!");
  • Location: ./libs/config/src/generate-json-schema.rs:50
  1. Reason: "part of development docgen CLI"
  • Code: .expect("Failed to write the json schema to the file system!");
  • Location: ./libs/config/src/generate-json-schema.rs:54
  1. Reason: "👇"
  • Code: let raw_contents = read_to_string(file_path)
  • Location: ./libs/config/src/lib.rs:815
  1. Reason: "👇"
  • Code: panic!("Failed to interpolate config file, please resolve the above errors");
  • Location: ./libs/config/src/lib.rs:847
  1. Reason: "👇"
  • Code: parse_config_from_json(&config_string).expect("Failed to parse JSON config file")
  • Location: ./libs/config/src/lib.rs:854
  1. Reason: "👇"
  • Code: parse_config_from_yaml(&config_string).expect("Failed to parse YAML config file")
  • Location: ./libs/config/src/lib.rs:858
  1. Reason: "👇"
  • Code: _ => panic!("Unsupported config file extension"),
  • Location: ./libs/config/src/lib.rs:875
  1. Reason: "👇"
  • Code: None => panic!("Config file has no extension"),
  • Location: ./libs/config/src/lib.rs:878
  1. Reason: "statically defined regex pattern, we know it works ;)"
  • Code: .unwrap();
  • Location: ./libs/config/src/interpolate.rs:18

Crate: common

📊 Total Expected Usages: 1

expand details
  1. Reason: "we're parsing a statically defined constant, we know it works ;)"
  • Code: .unwrap()
  • Location: ./libs/common/src/graphql.rs:31

Crate: jwt_auth

📊 Total Expected Usages: 1

expand details
  1. Reason: "if initiating an http client fails, then we have to exit."
  • Code: let client = wasm_polyfills::create_http_client().build().unwrap();
  • Location: ./plugins/jwt_auth/src/jwks_provider.rs:49

Crate: cloudflare_worker

📊 Total Expected Usages: 4

expand details
  1. Reason: "it panics only if the header name is not valid, and we know it is."
  • Code: .unwrap()
  • Location: ./bin/cloudflare_worker/src/http_tracing.rs:20
  1. Reason: "it panics only if the URL source is not valid, and it's already validated before."
  • Code: let url = req.url().unwrap();
  • Location: ./bin/cloudflare_worker/src/http_tracing.rs:23
  1. Reason: "it only panics if we are not running in a CF context, should be safe."
  • Code: let cf_info = req.cf().unwrap();
  • Location: ./bin/cloudflare_worker/src/http_tracing.rs:27
  1. Reason: "unwraps only in special cases where "data:text" is used."
  • Code: let http_host = url.host().unwrap().to_string();
  • Location: ./bin/cloudflare_worker/src/http_tracing.rs:36

Crate: napi

📊 Total Expected Usages: 1

expand details
  1. Reason: "we need this"
  • Code: panic!("Exited process!")
  • Location: ./libs/napi/src/lib.rs:18

Crate: engine

📊 Total Expected Usages: 2

expand details
  1. Reason: "if we are unable to construct the endpoints and attach them onto the gateway's http server, we have to exit"
  • Code: Err(e) => panic!("failed to construct endpoint: {:?}", e),
  • Location: ./libs/engine/src/gateway.rs:158
  1. Reason: "we can safely index here, it's inside a test with constant defined fixtures."
  • Code: ConductorGateway::execute(request, &gw.routes[0].route_data).await
  • Location: ./libs/engine/src/gateway.rs:190

@renovate renovate bot force-pushed the renovate/rust-wasm-bindgen-monorepo branch from a873c67 to 7ac3c6f Compare April 8, 2024 12:20
Copy link
github-actions bot commented Apr 8, 2024

🐋 This PR was built and pushed to the following Docker images:

Docker Bake metadata
{
"conductor": {
  "buildx.build.ref": "builder-08d34d07-0069-48d5-b6eb-2e4f96bbb4d1/builder-08d34d07-0069-48d5-b6eb-2e4f96bbb4d10/6n0mnuigppckb33p5c8vxyb5t",
  "containerimage.config.digest": "sha256:1006d7a699b3ee6700f2ffa24e59b68d442229e3551307ef29b7563a5e7d23bd",
  "containerimage.descriptor": {
    "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
    "digest": "sha256:f6da3e637520889edc0bfac080bfcffe5b9a3ba67ce7fc5619840533b033b7da",
    "size": 902,
    "platform": {
      "architecture": "amd64",
      "os": "linux"
    }
  },
  "containerimage.digest": "sha256:f6da3e637520889edc0bfac080bfcffe5b9a3ba67ce7fc5619840533b033b7da",
  "image.name": "ghcr.io/the-guild-org/conductor/conductor:7ac3c6fa9bb294e8ee0813740fb318f60db98f40"
}
}

< 8000 details-menu class="dropdown-menu dropdown-menu-sw show-more-popover color-fg-default" style="width:185px" src="" preload > Copy link
github-actions bot commented Apr 8, 2024

✅ Benchmark Results

     data_received..................: 13 MB   221 kB/s
     data_sent......................: 22 MB   363 kB/s
     http_req_blocked...............: min=1.08µs   avg=2.9µs    med=2.19µs   max=1.02ms   p(95)=3.16µs   p(99)=11.76µs 
     http_req_connecting............: min=0s       avg=354ns    med=0s       max=990.11µs p(95)=0s       p(99)=0s      
     http_req_duration..............: min=310.37µs avg=407.84µs med=379.18µs max=24.47ms  p(95)=477.71µs p(99)=559.62µs
       { expected_response:true }...: min=310.37µs avg=407.84µs med=379.18µs max=24.47ms  p(95)=477.71µs p(99)=559.62µs
     ✓ { scenario:rps_1000 }........: min=310.37µs avg=407.84µs med=379.18µs max=24.47ms  p(95)=477.71µs p(99)=559.62µs
     http_req_failed................: 0.00%   ✓ 0           ✗ 60002
     ✓ { scenario:rps_1000 }........: 0.00%   ✓ 0           ✗ 60002
     http_req_receiving.............: min=9.34µs   avg=25.97µs  med=25.33µs  max=964.03µs p(95)=34.33µs  p(99)=45.86µs 
     http_req_sending...............: min=6.87µs   avg=15.26µs  med=13.98µs  max=1.35ms   p(95)=25.21µs  p(99)=33.16µs 
     http_req_tls_handshaking.......: min=0s       avg=0s       med=0s       max=0s       p(95)=0s       p(99)=0s      
     http_req_waiting...............: min=275µs    avg=366.6µs  med=338.69µs max=24.42ms  p(95)=438.82µs p(99)=510.84µs
     http_reqs......................: 60002   1000.004585/s
     ✓ { scenario:rps_1000 }........: 60002   1000.004585/s
     iteration_duration.............: min=380.31µs avg=493.52µs med=463.54µs max=24.75ms  p(95)=565.89µs p(99)=702.04µs
     iterations.....................: 60002   1000.004585/s
     ✓ { scenario:rps_1000 }........: 60002   1000.004585/s
     valid_graphql_response.........: 100.00% ✓ 60002       ✗ 0    
     ✓ { scenario:rps_1000 }........: 100.00% ✓ 60002       ✗ 0    
     valid_http_code................: 100.00% ✓ 60002       ✗ 0    
     ✓ { scenario:rps_1000 }........: 100.00% ✓ 60002       ✗ 0    
     vus............................: 1       min=0         max=2  
     vus_max........................: 200     min=200       max=200

@dotansimha dotansimha merged commit 80808db into master Apr 8, 2024
@renovate renovate bot deleted the renovate/rust-wasm-bindgen-monorepo branch April 8, 2024 12:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0