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

Shorten hash #61

Closed
GoogleCodeExporter opened this issue Apr 6, 2015 · 2 comments
Closed

Shorten hash #61

GoogleCodeExporter opened this issue Apr 6, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

@GoogleCodeExporter GoogleCodeExporter commented Apr 6, 2015

The hash is currently 32-char long which allows for 64^32 == 2^192 
possibilities. This seems like overkill, we should be fine with 10-char -> 2^60 
by default and allow it to be configurable.

Original issue reported on code.google.com by sligocki@google.com on 9 Nov 2010 at 7:48

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Apr 6, 2015

We should indeed make the choice of hash strength configurable in some fashion, 
especially if we add a hash of the url to each rewritten url (which we might 
want to be shorter; we're not hashing that much data).

Original comment by jmaes...@google.com on 9 Nov 2010 at 8:00

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Apr 6, 2015

Original comment by jmara...@google.com on 3 Dec 2010 at 10:48

  • Changed state: Fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.