Add test and fix for panic in Bearer::to_bytes

actix / actix-web-httpauth

Moved to actix-extras repo.

https://github.com/actix/actix-extras/tree/HEAD/actix-web-httpauth

Apache License 2.0

82 stars 37 forks source link

Add test and fix for panic in Bearer::to_bytes #1

Closed griff closed 6 years ago

griff commented 6 years ago

There was a bug in the BufferMut::reserve logic in headers::www_authenticate::challenge::bearer::Bearer::to_bytes that would cause a panic.

This small commit adds a test that will panic with the old logic and it fixes the problem.

svartalf commented 6 years ago

@griff thanks for fix!

Do you need this at crates.io asap?

griff commented 6 years ago

Right now I am fine since I just use my own repo for the dependency. But its a pretty bad bug given that it panics so you might want to release it soon.

svartalf commented 6 years ago

@griff fyi, 0.0.4 version is now available at crates.io with your fix. Thanks again :)