Skip to content

Group redirect thing#573

Open
VinyarionHyarmendacil wants to merge 4 commits intovrchatapi:mainfrom
VinyarionHyarmendacil:group-redirect-header
Open

Group redirect thing#573
VinyarionHyarmendacil wants to merge 4 commits intovrchatapi:mainfrom
VinyarionHyarmendacil:group-redirect-header

Conversation

@VinyarionHyarmendacil
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@C0D3-M4513R C0D3-M4513R left a comment

Choose a reason for hiding this comment

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

The code generated in rust wouldn't return the redirected-to url.
Apparently in C# there's issues too.

idk about other sdk's

Generated Rust code
pub async fn resolve_group_short_code_and_discriminator(
    configuration: &configuration::Configuration,
    group_short_code_and_discriminator: &str,
) -> Result<(), Error<ResolveGroupShortCodeAndDiscriminatorError>> {
    // add a prefix to parameters to efficiently prevent name collisions
    let p_path_group_short_code_and_discriminator = group_short_code_and_discriminator;

    let uri_str = format!(
        "{}/groups/redirect/{groupShortCodeAndDiscriminator}",
        configuration.base_path,
        groupShortCodeAndDiscriminator =
            crate::apis::urlencode(p_path_group_short_code_and_discriminator)
    );
    let mut req_builder = configuration.client.request(reqwest::Method::GET, &uri_str);

    if let Some(ref user_agent) = configuration.user_agent {
        req_builder = req_builder.header(reqwest::header::USER_AGENT, user_agent.clone());
    }

    let req = req_builder.build()?;
    let resp = configuration.client.execute(req).await?;

    let status = resp.status();

    if !status.is_client_error() && !status.is_server_error() {
        Ok(())
    } else {
        let content = resp.text().await?;
        let entity: Option<ResolveGroupShortCodeAndDiscriminatorError> =
            serde_json::from_str(&content).ok();
        Err(Error::ResponseError(ResponseContent {
            status,
            content,
            entity,
        }))
    }
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants