Bug 2211432 (review-rust-ureq)

Summary:	Review Request: rust-ureq - Simple, safe HTTP client
Product:	[Fedora] Fedora	Reporter:	Michel Alexandre Salim <michel>
Component:	Package Review	Assignee:	Fabio Valentini <decathorpe>
Status:	ASSIGNED ---	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	decathorpe, package-review
Target Milestone:	---	Flags:	decathorpe: fedora-review?
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	2211172

Description Michel Alexandre Salim 2023-05-31 15:38:30 UTC

Spec URL: https://salimma.fedorapeople.org/rust-ureq.spec
SRPM URL: https://salimma.fedorapeople.org/rust-ureq-2.6.2-1.fc39.src.rpm

Description:
Simple, safe HTTP client.

Fedora Account System Username: salimma

Comment 1 Fabio Valentini 2023-05-31 18:10:53 UTC

Please disable all rustls related functionality and make native-tls the default tls backend.
Otherwise this package will be unusable on half our supported architectures.

Comment 2 Michel Alexandre Salim 2023-06-08 18:26:55 UTC

Ah, thanks! I'll look into this

Comment 3 Fabio Valentini 2023-07-29 14:15:16 UTC

Note that rustls dependencies + features can now theoretically remain in the package, they just can't be part of the default feature set. I still recommend that you remove the rustls related features (because you won't be able to use them on all architectures), but it's no longer a hard requirement.

The rustls packages are now available on all architectures, they just don't work on some of them (this should make packaging things that depend on them easier).

Comment 4 Fabio Valentini 2023-07-30 17:31:50 UTC

BTW, version 2.7.1 has been released in the meantime, it would be great if this could be updated to a newer version.
Note that the cookie_store crate in Fedora is currently too old for ureq (at v0.16, ureq needs v0.19).

If you need cookie related functionality, I can work on updating the cookie_store crate, otherwise I'd recommend just disabling features that depend on cookie_store v0.19 for now.