webpki icon indicating copy to clipboard operation
webpki copied to clipboard

Optional serde support for DnsName/DnsNameRef

Open clarfonthey opened this issue 3 years ago • 1 comments

I would add implement Serialize/Display for DnsNameRef but I figured there was a good reason to not store it as &str, and didn't want to bother with refactoring that without talking to anyone.

This adds serde support for both types under an optional feature. It also implements Display for DnsName since I figured it'd be reasonable to have.

clarfonthey avatar Aug 04 '21 23:08 clarfonthey

cc @briansmith as I'm going through old PRs: assuming I resolve the merge conflict, would you still be interested in merging this, or should I assume that inaction here means it's not desired? I don't mind either way, just want to check since this PR is 3 years old and I wouldn't mind rebasing it if you do think it's useful.

clarfonthey avatar Mar 24 '24 23:03 clarfonthey