Add `Ferrum::Cookies#each`

rubycdp / ferrum

Headless Chrome Ruby API

https://ferrum.rubycdp.com

MIT License

1.74k stars 123 forks source link

Add `Ferrum::Cookies#each` #367

Closed postmodern closed 1 year ago

postmodern commented 1 year ago

Added Ferrum::Cookies::Cookie#each.
Also make Ferrum::Cookies Enumerable.
Refactored Ferrum::Cookies#all to use #each.
Optimize Ferrum::Cookies#[] to use find { |cookie| ... }.

postmodern commented 1 year ago

The motivation for this PR is to allow selecting cookies by with the same name but different Path or Domain values. cookies.all would combine all cookies into a Hash, so cookies with the same name but different Domain would overwrite each other.

route commented 1 year ago

Perfect! Thank you!