Skip to content

Update deps for OpenIdConnect and Reqwest#593

Merged
buehler merged 21 commits intosmartive:mainfrom
fcenedes:update_deps
Mar 24, 2025
Merged

Update deps for OpenIdConnect and Reqwest#593
buehler merged 21 commits intosmartive:mainfrom
fcenedes:update_deps

Conversation

@fcenedes
Copy link
Copy Markdown
Contributor

@fcenedes fcenedes commented Feb 7, 2025

This PR updates openidconnect to 4.0.0 and Reqwest to his latest.
It required to change a couple of things. but all tests are ok.

it's based on the previous PR splitting into 2 crates
it should close #589 , #591 , #539

@buehler
Copy link
Copy Markdown
Collaborator

buehler commented Mar 18, 2025

Hey @fcenedes
Did you remove the github workflows? somehow I cannot start the workflow

@fcenedes
Copy link
Copy Markdown
Contributor Author

i dont think so : https://github.com/fcenedes/zitadel-rust/tree/update_deps/.github/workflows

havent touch that part

@fcenedes
Copy link
Copy Markdown
Contributor Author

Try to approve it now

@fcenedes
Copy link
Copy Markdown
Contributor Author

before publishing, you may want to review the strategy, the zitadel crate can follow your versioning scheme, while having a separate job to publish a matrix of supported zitadel-gen version (should we call this sys ?) (passing the version to just zitadel_proto_version var)

sorry if am unclear

@fcenedes
Copy link
Copy Markdown
Contributor Author

fcenedes commented Mar 24, 2025

@buehler : forgot clippy. should pass now :-)

@buehler buehler enabled auto-merge (squash) March 24, 2025 21:44
@buehler buehler merged commit e2b20a1 into smartive:main Mar 24, 2025
1 check passed
sprudel added a commit that referenced this pull request Mar 17, 2026
* Reverts crate split of
#593 making crate
releasable again
* added pipeline job for validating packaging works to prevent
non-releasable crate on main
* aligned crypto lib to aws_lc_rs where possible (reqwest via
openidconnect still uses ring)
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