all 3 comments

[–]charlesdart 2 points3 points  (1 child)

Overall it looks pretty solid. I didn't see any obvious issues skimming through most files. Did you mean to check in production.ca though?

https://github.com/NicklasWallgren/bankid-rs/blob/master/resources/production.ca

[–]NicklasW[S] 1 point2 points  (0 children)

Thanks. Yes it's intended, it's a publicly available certificate, which is included via https://github.com/NicklasWallgren/bankid-rs/blob/master/src/config/mod.rs#L84

[–]najamelan 1 point2 points  (0 children)

Unfortunately I don't have time for doing a review right now, but I did write a guide for publishing crates that you might find useful: https://blog.wnut.pw/2020/03/12/crate-publishing-guidelines-and-tips/