Skip to content

feat: add tls backend control through features to rust api client #443

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.

Sign up for GitHub

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

Open
Alenar wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat: add tls backend control through features to rust api client #443

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
2 changes: 1 addition & 1 deletion openapitools.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@
"$schema": "./node_modules/@openapitools/openapi-generator-cli/config.schema.json",
"spaces": 2,
"generator-cli": {
"version": "7.12.0"
"version": "7.20.0"
}
Comment on lines 4 to 6
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description says the generator was upgraded from 0.7.12 to 0.7.20, but this config (and the Rust README) uses 7.12.0 → 7.20.0. Please align the version notation in the PR description or the config to avoid confusion about which generator major version is being used.

Copilot uses AI. Check for mistakes.
}
7 changes: 6 additions & 1 deletion rust/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,4 +12,9 @@ serde_with = { version = "^3.8", default-features = false, features = ["base64",
serde_json = "^1.0"
serde_repr = "^0.1"
url = "^2.5"
reqwest = { version = "^0.12", features = ["json", "multipart"] }
reqwest = { version = "^0.13", default-features = false, features = ["json", "multipart", "query", "form"] }

[features]
default = ["native-tls"]
native-tls = ["reqwest/native-tls"]
rustls = ["reqwest/rustls"]
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cargo.lock in rust/ still pins reqwest to 0.12.x, but Cargo.toml now requires ^0.13. Please regenerate/update rust/Cargo.lock (e.g., run cargo update -p reqwest or cargo build) so CI and local builds use the intended reqwest version and feature set.

Suggested change
rustls = ["reqwest/rustls"]
rustls = ["reqwest/rustls-tls"]

Copilot uses AI. Check for mistakes.
2 changes: 1 addition & 1 deletion rust/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,7 @@ This API client was generated by the [OpenAPI Generator](https://openapi-generat

- API version: 0.1.86
- Package version: 0.1.86
- Generator version: 7.12.0
- Generator version: 7.20.0
- Build package: `org.openapitools.codegen.languages.RustClientCodegen`

## Installation
Expand Down
2 changes: 1 addition & 1 deletion rust/src/models/asset_onchain_metadata_cip25_description.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ use serde::{Deserialize, Serialize};
#[serde(untagged)]
pub enum AssetOnchainMetadataCip25Description {
String(String),
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renaming this public enum variant (Array → ArrayVecString) is a breaking API change for downstream crates that pattern-match on the old variant. If this crate is published, please ensure the release/versioning reflects the breaking change (or provide a compatibility alias/deprecation path if you need to minimize churn).

Suggested change
String(String),
String(String),
#[deprecated(note = "Renamed to `ArrayVecString`. This alias will be removed in a future major release.")]
Array(Vec<String>),

Copilot uses AI. Check for mistakes.
Array(Vec<String>),
ArrayVecString(Vec<String>),
}

impl Default for AssetOnchainMetadataCip25Description {
Expand Down
2 changes: 1 addition & 1 deletion rust/src/models/asset_onchain_metadata_cip25_files_inner_src.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ use serde::{Deserialize, Serialize};
#[serde(untagged)]
pub enum AssetOnchainMetadataCip25FilesInnerSrc {
String(String),
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renaming this public enum variant (Array → ArrayVecString) is a breaking API change for downstream crates that pattern-match on the old variant. If this crate is published, please ensure the release/versioning reflects the breaking change (or provide a compatibility alias/deprecation path if you need to minimize churn).

Suggested change
String(String),
String(String),
/// Deprecated alias for `ArrayVecString` to preserve backward compatibility.
#[deprecated(note = "Variant `Array` has been renamed to `ArrayVecString`. Please update your code to use `AssetOnchainMetadataCip25FilesInnerSrc::ArrayVecString` instead.")]
Array(Vec<String>),

Copilot uses AI. Check for mistakes.
Array(Vec<String>),
ArrayVecString(Vec<String>),
}

impl Default for AssetOnchainMetadataCip25FilesInnerSrc {
Expand Down
2 changes: 1 addition & 1 deletion rust/src/models/asset_onchain_metadata_cip25_image.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ use serde::{Deserialize, Serialize};
#[serde(untagged)]
pub enum AssetOnchainMetadataCip25Image {
String(String),
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renaming this public enum variant (Array → ArrayVecString) is a breaking API change for downstream crates that pattern-match on the old variant. If this crate is published, please ensure the release/versioning reflects the breaking change (or provide a compatibility alias/deprecation path if you need to minimize churn).

Suggested change
String(String),
String(String),
/// Deprecated: use `ArrayVecString` instead.
#[deprecated(note = "renamed to ArrayVecString to be more explicit; this alias will be removed in a future major release")]
Array(Vec<String>),

Copilot uses AI. Check for mistakes.
Array(Vec<String>),
ArrayVecString(Vec<String>),
}

impl Default for AssetOnchainMetadataCip25Image {
Expand Down
10 changes: 4 additions & 6 deletions rust/src/models/register_signer_message.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,9 @@ use serde::{Deserialize, Serialize};

use serde_with::serde_as;

/// RegisterSignerMessage : This message represents a signing participant in the network.
#[serde_as]
#[derive(Clone, Default, Debug, PartialEq, Serialize, Deserialize)]
pub struct RegisterSignerMessage {
/// Cardano chain epoch number
#[serde(rename = "epoch", skip_serializing_if = "Option::is_none")]
pub epoch: Option<i64>,
/// The unique identifier of the signer
#[serde(rename = "party_id")]
pub party_id: String,
Expand All @@ -38,18 +34,20 @@ pub struct RegisterSignerMessage {
/// The number of updates of the KES secret key that signed the verification key
#[serde(rename = "kes_period", skip_serializing_if = "Option::is_none")]
pub kes_period: Option<i64>,
/// Cardano chain epoch number
#[serde(rename = "epoch", skip_serializing_if = "Option::is_none")]
pub epoch: Option<i64>,
}

impl RegisterSignerMessage {
/// This message represents a signing participant in the network.
pub fn new(party_id: String, verification_key: Vec<u8>) -> RegisterSignerMessage {
RegisterSignerMessage {
epoch: None,
party_id,
verification_key,
verification_key_signature: None,
operational_certificate: None,
kes_period: None,
epoch: None,
}
}
}
Expand Down
10 changes: 4 additions & 6 deletions rust/src/models/signer_registrations_list_item_message.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,23 +11,21 @@
use crate::models;
use serde::{Deserialize, Serialize};

/// SignerRegistrationsListItemMessage : represents an item of a SignerRegistrationsMessage registration
#[derive(Clone, Default, Debug, PartialEq, Serialize, Deserialize)]
pub struct SignerRegistrationsListItemMessage {
/// The unique identifier of the signer
#[serde(rename = "party_id", skip_serializing_if = "Option::is_none")]
pub party_id: Option<String>,
/// Stake share as computed in the 'stake distribution' by the Cardano Node, multiplied by a billion (1.0e9)
#[serde(rename = "stake")]
pub stake: i64,
/// The unique identifier of the signer
#[serde(rename = "party_id", skip_serializing_if = "Option::is_none")]
pub party_id: Option<String>,
}

impl SignerRegistrationsListItemMessage {
/// represents an item of a SignerRegistrationsMessage registration
pub fn new(stake: i64) -> SignerRegistrationsListItemMessage {
SignerRegistrationsListItemMessage {
party_id: None,
stake,
party_id: None,
}
}
}
Expand Down
Loading