Skip to content

Group redirect thing - #573

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

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

Conversation

@VinyarionHyarmendacil

Copy link
Copy Markdown
Contributor

No description provided.

@C0D3-M4513RC0D3-M4513R left a comment

Copy link
Copy Markdown
Contributor

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
pubasyncfnresolve_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 collisionslet 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));letmut req_builder = configuration.client.request(reqwest::Method::GET,&uri_str);ifletSome(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 freeto 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

@VinyarionHyarmendacil@C0D3-M4513R