8000 chore(deps): update rust crate async-trait to 0.1.77 by renovate[bot] · Pull Request #324 · 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.

chore(deps): update rust crate async-trait to 0.1.77 #324

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

renovate[bot]
Copy link
Contributor
@renovate renovate bot commented Jan 11, 2024

Mend Renovate

This PR contains the following updates:

Package Type Update Change
async-trait workspace.dependencies patch 0.1.74 -> 0.1.77

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 Jan 11, 2024

🚨 Rust Panic Audit: 59 Potential Panic Points Detected 🚨

Crate: federation_query_planner

📊 Total Usages: 50

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

Crate: engine

📊 Total Usages: 9

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

📌 Expected Annotations

Crate: engine

📊 Total Expected Usages: 3

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:113
  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:144
  1. Reason: "without a fetcher, there's no executor, without an executor, there's no gateway."
  • Code: panic!(
  • Location: ./libs/engine/src/source/graphql_source.rs:28

Crate: disable_introspection

📊 Total Expected Usages: 1

expand details
  1. Reason: "we need to exit the process if our provided VRL condition has incorrect syntax."
  • Code: panic!("failed to compile vrl program for disable_introspection plugin");
  • Location: ./plugins/disable_introspection/src/plugin.rs:33

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: config

📊 Total Expected Usages: 9

expand details
  1. Reason: "statically defined regex pattern, we know it works ;)"
  • Code: .unwrap();
  • Location: ./libs/config/src/interpolate.rs:18
  1. Reason: "👇"
  • Code: let raw_contents = read_to_string(file_path).expect("Failed to read config file");
  • Location: ./libs/config/src/lib.rs:415
  1. Reason: "👇"
  • Code: panic!("Failed to interpolate config file, please resolve the above errors");
  • Location: ./libs/config/src/lib.rs:445
  1. Reason: "👇"
  • Code: parse_config_from_json(&config_string).expect("Failed to parse JSON config file")
  • Location: ./libs/config/src/lib.rs:452
  1. Reason: "👇"
  • Code: parse_config_from_yaml(&config_string).expect("Failed to parse YAML config file")
  • Location: ./libs/config/src/lib.rs:456
  1. Reason: "👇"
  • Code: _ => panic!("Unsupported config file extension"),
  • Location: ./libs/config/src/lib.rs:473
  1. Reason: "👇"
  • Code: None => panic!("Config file has no extension"),
  • Location: ./libs/config/src/lib.rs:476
  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

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:22

Crate: vrl

📊 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:122
  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:139

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:47

Crate: conductor

📊 Total Expected Usages: 3

expand details
  1. Reason: "we need to exit the process, if the logger can't be correctly set."
  • Code: tracing 8000 ::subscriber::set_global_default(subscriber).expect("failed to set up the logger");
  • Location: ./bin/conductor/src/lib.rs:30
  1. Reason: "we need to exit, if the provided log level in the configuration file is incompaitable."
  • Code: .expect("Failed to modify the log level");
  • Location: ./bin/conductor/src/lib.rs:45
  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:80

@renovate renovate bot force-pushed the renovate/async-trait-0.x branch from bf3e240 to 0f23f62 Compare January 11, 2024 15:56
@dotansimha dotansimha merged commit 41a0e6c into master Jan 11, 2024
@dotansimha dotansimha deleted the renovate/async-trait-0.x branch January 11, 2024 15:57
Copy link
github-actions bot commented Jan 11, 2024

✅ Benchmark Results

     data_received..................: 13 MB   221 kB/s
     data_sent......................: 22 MB   363 kB/s
     http_req_blocked...............: min=1.05µs   avg=3.08µs   med=2.36µs   max=990.95µs p(95)=3.28µs  p(99)=9.7µs  
     http_req_connecting............: min=0s       avg=356ns    med=0s       max=527.72µs p(95)=0s      p(99)=0s     
     http_req_duration..............: min=777.45µs avg=1ms      med=907.6µs  max=50.32ms  p(95)=1.29ms  p(99)=1.82ms 
       { expected_response:true }...: min=777.45µs avg=1ms      med=907.6µs  max=50.32ms  p(95)=1.29ms  p(99)=1.82ms 
     ✓ { scenario:rps_1000 }........: min=777.45µs avg=1ms      med=907.6µs  max=50.32ms  p(95)=1.29ms  p(99)=1.82ms 
     http_req_failed................: 0.00%   ✓ 0          ✗ 60001
     ✓ { scenario:rps_1000 }........: 0.00%   ✓ 0          ✗ 60001
     http_req_receiving.............: min=9.56µs   avg=29.63µs  med=27.79µs  max=1.67ms   p(95)=46.15µs p(99)=58.04µs
     http_req_sending...............: min=6.95µs   avg=15.9µs   med=13.8µs   max=1.57ms   p(95)=30.05µs p(99)=38.74µs
     http_req_tls_handshaking.......: min=0s       avg=0s       med=0s       max=0s       p(95)=0s      p(99)=0s     
     http_req_waiting...............: min=741.35µs avg=961.55µs med=861.78µs max=50.25ms  p(95)=1.24ms  p(99)=1.76ms 
     http_reqs......................: 60001   999.981205/s
     ✓ { scenario:rps_1000 }........: 60001   999.981205/s
     iteration_duration.............: min=855.55µs avg=1.1ms    med=1ms      max=50.56ms  p(95)=1.38ms  p(99)=1.94ms 
     iterations.....................: 60001   999.981205/s
     ✓ { scenario:rps_1000 }........: 60001   999.981205/s
     valid_graphql_response.........: 100.00% ✓ 60001      ✗ 0    
     ✓ { scenario:rps_1000 }........: 100.00% ✓ 60001      ✗ 0    
     valid_http_code................: 100.00% ✓ 60001      ✗ 0    
     ✓ { scenario:rps_1000 }........: 100.00% ✓ 60001      ✗ 0    
     vus............................: 2       min=0        max=3  
     vus_max........................: 200     min=200      max=200

YassinEldeeb pushed a commit that referenced this pull request Jan 12, 2024
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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