8000 chore(deps): update pnpm to v8.15.5 by renovate[bot] · Pull Request #515 · 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 pnpm to v8.15.5 #515

Merged
merged 1 commit into from
Mar 20, 2024
Merged

chore(deps): update pnpm to v8.15.5 #515

merged 1 commit into from
Mar 20, 2024

Conversation

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

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
pnpm (source) 8.15.4 -> 8.15.5 age adoption passing confidence

Release Notes

pnpm/pnpm (pnpm)

v8.15.5

Compare Source


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 17, 2024

🚨 Rust Panic Audit: 87 Potential Panic Points Detected 🚨

Crate: federation_query_planner

📊 Total Usages: 51

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

Crate: engine

📊 Total Usages: 11

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

Crate: telemetry

📊 Total Usages: 9

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

Crate: cloudflare_worker

📊 Total Usages: 6

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

Crate: tracing

📊 Total Usages: 6

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

Crate: conductor

📊 Total Usages: 3

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

Crate: config

📊 Total Usages: 1

  • 🚨 panic usages: 1

📌 Expected Annotations

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

📊 Total Expected Usages: 9

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

Crate: engine

📊 Total Expected Usages: 5

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:147
  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:179
  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:30
  1. Reason: "without a fetcher, there's no executor, without an executor, there's no gateway."
  • Code: panic!("Failed while initializing the executor's fetcher for Federation source");
  • Location: ./libs/engine/src/source/federation_source.rs:111
  1. Reason: "without a fetcher, there's no executor, without an executor, there's no gateway."
  • Code: panic!("Failed while initializing the executor's fetcher for Federation source");
  • Location: ./libs/engine/src/source/federation_source.rs:142

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: tracing::subscriber::set_global_default(subscriber).expect("failed to set up tracing");
  • 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: 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

Copy link
github-actions bot commented Mar 17, 2024

💻 Website Preview

The latest changes are available as preview in: https://73868f93.conductor-t2.pages.dev

Copy link
github-actions bot commented Mar 17, 2024

✅ Benchmark Results

     data_received..................: 13 MB   221 kB/s
     data_sent......................: 22 MB   363 kB/s
     http_req_blocked...............: min=922ns    avg=2.79µs   med=2.09µs   max=553.57µs p(95)=3.01µs   p(99)=7.33µs  
     http_req_connecting............: min=0s       avg=380ns    med=0s       max=500.07µs p(95)=0s       p(99)=0s      
     http_req_duration..............: min=311.49µs avg=407.42µs med=382.86µs max=25.57ms  p(95)=469.07µs p(99)=549.87µs
       { expected_response:true }...: min=311.49µs avg=407.42µs med=382.86µs max=25.57ms  p(95)=469.07µs p(99)=549.87µs
     ✓ { scenario:rps_1000 }........: min=311.49µs avg=407.42µs med=382.86µs max=25.57ms  p(95)=469.07µs p(99)=549.87µs
     http_req_failed................: 0.00%   ✓ 0          ✗ 60000
     ✓ { scenario:rps_1000 }........: 0.00%   ✓ 0          ✗ 60000
     http_req_receiving.............: min=9.6µs    avg=25.84µs  med=25.35µs  max=433.46µs p(95)=33.8µs   p(99)=43.14µs 
     http_req_sending...............: min=6.76µs   avg=15.16µs  med=13.88µs  max=1.97ms   p(95)=24.67µs  p(99)=32.6µs  
     http_req_tls_handshaking.......: min=0s       avg=0s       med=0s       max=0s       p(95)=0s       p(99)=0s      
     http_req_waiting...............: min=272.93µs avg=366.41µs med=342.13µs max=25.52ms  p(95)=428.18µs p(99)=504.35µs
     http_reqs......................: 60000   999.987002/s
     ✓ { scenario:rps_1000 }........: 60000   999.987002/s
     iteration_duration.............: min=383.51µs avg=493.47µs med=467.08µs max=25.81ms  p(95)=557.92µs p(99)=724.86µs
     iterations.....................: 60000   999.987002/s
     ✓ { scenario:rps_1000 }........: 60000   999.987002/s
     valid_graphql_response.........: 100.00% ✓ 60000      ✗ 0    
     ✓ { scenario:rps_1000 }........: 100.00% ✓ 60000      ✗ 0    
     valid_http_code................: 100.00% ✓ 60000      ✗ 0    
     ✓ { scenario:rps_1000 }........: 100.00% ✓ 60000      ✗ 0    
     vus............................: 1       min=0        max=2  
     vus_max........................: 200     min=200      max=200

@renovate renovate bot force-pushed the renovate/pnpm-8.x branch from acdbc8f to 0899a8d Compare March 18, 2024 08:29
@renovate renovate bot force-pushed the renovate/pnpm-8.x branch from 0899a8d to e5b67c0 Compare March 19, 2024 13:49
Copy link

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

Docker Bake metadata
{
"conductor": {
  "buildx.build.ref": "builder-b7fa879b-03d0-4ecf-958d-fe1ed988d199/builder-b7fa879b-03d0-4ecf-958d-fe1ed988d1990/ywoe78v0adci7ccpjd6c6pb14",
  "containerimage.config.digest": "sha256:f4e777fdf8cc0b4ed29143405a495178c37dcbdeeeebcc6416fc13800049bf6b",
  "containerimage.descriptor": {
    "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
    "digest": "sha256:31ca205377af9a6f4510496fad1eb1efa4694d6e340906aafacab5f545ce1182",
    "size": 902,
    "platform": {
      "architecture": "amd64",
      "os": "linux"
    }
  },
  "containerimage.digest": "sha256:31ca205377af9a6f4510496fad1eb1efa4694d6e340906aafacab5f545ce1182",
  "image.name": "ghcr.io/the-guild-org/conductor/conductor:e5b67c0c05e4e001561ddf47837c6e60f900464a"
}
}

@dotansimha dotansimha merged commit 4889134 into master Mar 20, 2024
@renovate renovate bot 615C deleted the renovate/pnpm-8.x branch March 20, 2024 15:37
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