Address columns by name instead of id

browserutils / kooky

Go code to read cookies from browser cookie stores.

MIT License

210 stars 41 forks source link

Address columns by name instead of id #19

Closed srlehn closed 4 years ago

srlehn commented 4 years ago

The table layout of the firefox db did change a few times in the past. I don't expect the ids to be stable. With this PR we look columns up by name instead in firefox and chrome.

zellyn commented 4 years ago

This looks good. It might make sense to pull the "get column names" functionality into a helper, but that could be a separate PR.