Should we rewrite the existing get and head in terms of a session? This is how requests implements its module-level request methods, might make sense to take inspiration from that.

A

It's probably a good idea to re-use the same code paths for both methods.. my rationale when duplicating the implementations was to avoid modifying any non-linkcheck-builder code.

I'll take another look into consolidating them into shared implementations.

What do you think of the approach in b7d7fcf ?

I've pushed a commit to only implement the lower-level Session.request, to avoid duplication -- hopefully all looks good!

A

I've pushed a commit to only implement the lower-level Session.request, to avoid duplication -- hopefully all looks good!

Nice - yep, that looks much better to me 👍

Thanks @jayaddison -- this is the culmination of quite a bit of preparatory work, thank you for driving this all through!

A

Thank you very much @AA-Turner :)

You're welcome. I'll follow along with the next release to listen for user feedback.