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.

Sign up for GitHub

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

Jump to bottom

[FR] Use HEAD instead of GET in UnshortenLink #506

Closed
jeromeleonard opened this issue Jun 29, 2019 · 0 comments
Closed

[FR] Use HEAD instead of GET in UnshortenLink #506

jeromeleonard opened this issue Jun 29, 2019 · 0 comments
Labels
category:enhancement Issue is related to an existing feature to improve scope:analyzer Issue is analyzer related
Milestone
2.2.0

Comments

@jeromeleonard
Copy link
Contributor

Feature description
UnshortenLink does a GET request on the URL data given as input and waits for a 301/302 to extract the Location header.

Describe the solution you'd like
Do a HEAD request should be enough to get the Location if the HTTP response is 301/302.
@jeromeleonard jeromeleonard added category:enhancement Issue is related to an existing feature to improve scope:analyzer Issue is analyzer related labels Jun 29, 2019
@jeromeleonard jeromeleonard added this to the 2.2.0 milestone Jun 29, 2019
jeromeleonard added a commit that referenced this issue Jun 29, 2019
@jeromeleonard
#506 upgrade to v 1.2. Use HEAD instead of GET request
5996658
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
category:enhancement Issue is related to an existing feature to improve scope:analyzer Issue is analyzer related
Projects
None yet
Milestone
2.2.0
Development

No branches or pull requests
1 participant
@jeromeleonard