37
you are viewing a single comment's thread
view the rest of the comments
[-] MooseBoys@lemmy.world 2 points 1 year ago* (last edited 1 year ago)

Honestly url.join looks like a cluster-fuck of landmines to me: https://github.com/servo/rust-url/issues/333

I’d probably have just stuck with strings as well.

[-] BB_C@lemm.ee 1 points 1 year ago

I’d probably have just stuck with strings as well.

And this argument works as long as nothing wrong happens. Well, something wrong happened ;)

Smashing strings together is how this bug happened.

Constructing URLs reliably should have been the obvious first takeaway, was my point, instead of pretending the issue is not there. If Url::join() is somehow too confusing for some, then there are other ways to do it with simpler API, no problem.

this post was submitted on 28 Jul 2023
37 points (97.4% liked)

Rust

5953 readers
11 users here now

Welcome to the Rust community! This is a place to discuss about the Rust programming language.

Wormhole

!performance@programming.dev

Credits

  • The icon is a modified version of the official rust logo (changing the colors to a gradient and black background)

founded 1 year ago
MODERATORS