resolve now defaulting to HEAD method when applicable

Fix #854

docs/web.md

@@ -78,6 +78,7 @@ redirections = resolve("https://www.lemonde.fr/", stateful=True)
 *Arguments*
 
 - **url** *str*: url to request.
+- **method** *Optional[str]*: HTTP method to use when resolving. Default to `HEAD` if no body sniffing is required, else `GET`.
 - **headers** *Optional[dict[str, str]]*: HTTP headers, as a dict, to use for the request.
 - **cookie** *Optional[str | dict[str, str]]*: cookie string to pass as the `Cookie` header value. Can also be a cookie morsel dict mapping names to their values.
 - **spoof_ua** *bool* `False`: whether to use a plausible `User-Agent` header when performing the query.

minet/web.py

@@ -1096,7 +1096,7 @@ def request(
 def resolve(
     url: str,
     pool_manager: urllib3.PoolManager = DEFAULT_POOL_MANAGER,
-    method: str = "GET",
+    method: Optional[str] = None,
     headers: Optional[Dict[str, str]] = None,
     cookie: Optional[Union[str, Dict[str, str]]] = None,
     spoof_ua: bool = False,
@@ -1111,6 +1111,12 @@ def resolve(
     raise_on_statuses: Optional[Container[int]] = None,
     stateful: bool = False,
 ) -> RedirectionStack:
+    if method is None:
+        method = "HEAD"
+
+        if follow_meta_refresh or follow_js_relocation or canonicalize:
+            method = "GET"
+
     final_headers = build_request_headers(
         headers=headers, cookie=cookie, spoof_ua=spoof_ua
     )