Fix "domain" KeyError with Firefox .har

[hlohaus]

No description provided.

[github-actions]

Pull Request Review for "Fix 'domain' KeyError with Firefox .har"

Hello H Lohaus,

Thank you for your contribution to the project. Your pull request titled "Fix 'domain' KeyError with Firefox .har" has been reviewed. Below are the comments:

Code Review:

• The addition of the DOMAINS list is a good approach to handle domain matching.
• The refactoring in the read_cookie_files function improves the readability and efficiency of the code by removing the unnecessary check for the "domain" key in cookies.
• The implementation of the get_domain function is a clever way to extract the domain from the request headers.

Suggestions:

• It would be beneficial to include a fallback mechanism in the get_domain function in case the domain is not found in the DOMAINS list.
• Consider adding error handling for the case where get_domain returns None to avoid potential issues during runtime.

Overall, the changes are well thought out and address the KeyError issue effectively. Great work!

Best,
g4f Copilot