Fix error loading requests with empty header values from history

Closes #400

CHANGELOG.md

@@ -13,6 +13,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
   - [See docs](https://slumber.lucaspickering.me/book/troubleshooting/shell_completions.html) for more info and a list of supported shells
 - Add `slumber gen` alias to `--help` documentation
 
+### Fixed
+
+- Fix error loading requests with empty header values from history [#400](https://github.com/LucasPickering/slumber/issues/400)
+
 ## [2.1.0] - 2024-09-27
 
 ### Added

crates/core/src/db/convert.rs

@@ -302,6 +302,9 @@ impl<'a> ToSql for SqlWrap<&'a HeaderMap> {
 
 impl FromSql for SqlWrap<HeaderMap> {
     fn column_result(value: ValueRef<'_>) -> FromSqlResult<Self> {
+        // There's no easy way to re-use the header parsing logic from the http
+        // crate, so we have to reimplement it ourselves
+
         fn header_line(
             input: &mut &[u8],
         ) -> PResult<(HeaderName, HeaderValue)> {
@@ -312,7 +315,7 @@ impl FromSql for SqlWrap<HeaderMap> {
                     HEADER_FIELD_DELIM,
                 ),
                 terminated(
-                    take_while(1.., |c| c != HEADER_LINE_DELIM)
+                    take_while(0.., |c| c != HEADER_LINE_DELIM)
                         .try_map(HeaderValue::from_bytes),
                     HEADER_LINE_DELIM,
                 ),