Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unthrottle streams for all formats #244

Closed
wants to merge 1 commit into from

Conversation

elProxy
Copy link
Contributor

@elProxy elProxy commented Apr 24, 2022

Description

The n-param potentially requires processing even if there's no
deciphering needed.
Refactor decipherURL to reuse decryptNParam on any url.

Issues to Fix

#230

#232

Reminding

Something you can do before PR to reduce time to merge

  • run "make build" to build the code
  • run "make format" to reformat the code
  • run "make lint" if you are using unix system
  • run "make test-integration" to pass all tests

The n-param potentially requires processing even if there's no
deciphering needed.
Refactor decipherURL to reuse decryptNParam on any url.
@elProxy elProxy force-pushed the always-unthrottle-streams branch from 2540fb2 to 7f71515 Compare April 24, 2022 13:16
@elProxy
Copy link
Contributor Author

elProxy commented Apr 24, 2022

@corny

@corny
Copy link
Collaborator

corny commented Apr 25, 2022

Thanks your for work! I've just released a new version.

@corny corny closed this Apr 25, 2022
@elProxy
Copy link
Contributor Author

elProxy commented Apr 25, 2022

No problem. Thanks for all the work you guys put in maintaining this very handy library. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants