Skip to content

save_large_file should use default user-agent and accept arbitrary headers #360

Description

@benoit74

While #353 fixed stream_file, save_large_file has not been fixed as well.

The function would also benefit from accepting arbitrary headers.

Activity

  1. added this to the 5.5.0 milestone on Oct 22, 2026
  2. adityahabib commented on Oct 22, 2026

    @adityahabib

    Hi! I'd like to work on this issue. I'll look at how #353 fixed stream_file and apply the same approach to save_large_file, plus add support for custom headers. Is it okay if I take it?

  3. benoit74 commented on Oct 22, 2026

    @benoit74
    CollaboratorAuthor

    @adityahabib please push a PR

  4. added a commit that references this issue on Oct 22, 2026
    6b90e66
  5. benoit74 commented on Oct 22, 2026

    @benoit74
    CollaboratorAuthor

    @pawank925 I appreciate you moving fast, but pushing a PR to an issue already assigned to a volunteer which recently commit to work on it is a bit rude, looks like hijacking to me. I really want to maintain a welcoming community and give everyone a chance to contribute, no matter how fast or good they are (with some boundaries of course). Please do not close PR for now since work is done, but let's see where we are.

    @adityahabib where are you on implementation of this issue?

  6. pawank925 commented on Oct 22, 2026

    @pawank925
    Contributor

    @benoit74 really sorry previoulsy i worked on stream_file so i thought implementing save_large_file is an importnat one as well apolozies and i have worked one it 10 hrs again anyhow if you order i will close it @adityahabib sorry as well

    and i will be in control now ownwards sorry!

  7. benoit74 commented on Oct 22, 2026

    @benoit74
    CollaboratorAuthor

    You definitely deserve to be assigned this issue, I just want to avoid double work on same issue + not letting other people work at their pace ; I do really appreciate all the hard work you've already pushed in our community

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions