Victor Nordam Suadicani

Results 13 comments of Victor Nordam Suadicani

I just ran into this error in production. On large uploads, I deliberately stop reading the payload early if the user is already over the upload limit (why would I...

I've built a small wrapping library around `client_rust` for internal use at my job, to streamline and standardize our usage. Basically I'm not using a plain Histogram as this is...

Tangentially related, but I noticed that `&str` also implements encode - does that assume that you _must_ have the `&str` be equal to `"label_name=\"label_value\"`? I'm assuming that if you just...

I honestly find it a bit surprising that `&str` implements `Encode` in that case. I guess it's the same with `f32` and the other primitives that implement `Encode`. `encode` should...

I think the proper solution would be to weigh exact (case-sensitive) matches higher, but still find the other (case-insensitive) matches. As in, `rs Eq` would still find both `std::ptr::eq` and...

How would this interact with a problem like the one in #75? I mean, `()` also implements Serialize/Deserialize but you probably shouldn't be able to use `()` as a label,...

The problem I'm trying to solve is what is mentioned in the linked issues, such as https://github.com/rust-lang/docker-rust/issues/97 Basically, the problem currently is that the official Rust docker image doesn't have...

@massimiliano-mantione I think this sounds like an excellent idea, though I would love to also be able to get rid of the optionality entirely. This is useful for a server...

I think using custom field attributes would be really interesting as a much better alternative than `.type_attribute`. I don't think prost should use [PGV](https://github.com/envoyproxy/protoc-gen-validate) though, at least not yet. Firstly,...

I had basically the same thought in #699. The fact that prost doesn't allow for any extra validation leads to a lot of duplication of types that are basically the...