Support for custom HTTP Headers

[2moe]

Pull Request

Types of changes

• New feature (Support for custom HTTP Headers)
• Chore (Upgrade and simplify deps)

Description

close #52

About feature:

Added .headers() and .header() for DownloaderBuilder.
We can either call the .header() multiple times to add multiple headers, or we can add a bunch of headers with .headers().

   use reqwest::{
            header::{self, HeaderValue},
        };

    const FIREFOX_UA: &str =
        "Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/109.0";

    let ua = HeaderValue::from_str(FIREFOX_UA).expect("Invalid UA");

    let client = DownloaderBuilder::new()
        .directory(PathBuf::from("assets"))
        .header(header::USER_AGENT, ua)
        .build();

About dependencies:

http::header -> reqwest::header
url::Url -> reqwest::Url

Reducing external dependencies makes maintenance easier, as well as reducing the probability of dependency conflicts.

Checklist

• I have updated the documentation accordingly
• [] I have updated the Changelog (if applicable)

[rgreinho]

Looking good! Thank you for your contribution 👍

Only one minor thing I would like you to change regarding the version number.

Also, for some reason the formatting of the PR seems off and causes cargo fmt to complain. Are you using Rust nightly?

[rgreinho]

Don't update the version number. We change it when we cut a new release.

[2moe]

I am using rust stable. (rustc 1.66.1 (90743e729 2023-01-10))
The format is mainly related to rustfmt.

    "rust-analyzer.rustfmt.extraArgs": [
        "--config",
        "max_width=85,single_line_if_else_max_width=80,chain_width=32,format_code_in_doc_comments=false,imports_granularity=Crate"
    ],
    "rust-analyzer.checkOnSave.command": "clippy",

Could you provide the relevant configuration?

Ok, It has been reformatted according to the default configuration, rather than using the above configuration.

[rgreinho]

👍

[rgreinho]

👍