urls fail on links > 80 chars

[sckasturi]

e.g. http://37.media.tumblr.com/5bb8e8edbbcdfeb6a77b56eb9fda81aa/tumblr_n5sm2hdSkp1r0ciq0o1_500.jpg becomes http://37.media.tumblr.com/5bb8e8edbbcdfeb6a77b56eb9fda81aa/tumblr_n5sm2hdSkp1r0

[cydrobolt]

The issue pertained only to TempURLs, since the schema was imported from a previous version of Polr. Thanks @sckasturi for reporting this bug, it has been fixed. (limit is now 250 for tempurls)

Additional note: the temp table was truncated.

[sckasturi]

personally I would not set a limit as urls themselves dont have a limit.

Sent from my Samsung Galaxy S3
On May 21, 2014 3:29 PM, "Chaoyi Zha" notifications@github.com wrote:

The issue pertained only to TempURLs, since the schema was imported from a
previous version of Polr. Thanks @sckasturi https://github.com/sckasturifor reporting this bug, it has been fixed. (limit is now 250 for tempurls)

Additional note: the temp table was truncated.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/34#issuecomment-43802947
.

[cydrobolt]

I know this, but we can't have people DoSing our servers by making "urls" that are extremely long

[sckasturi]

then instead of only shortening the first 80 chars of the url I would
raise an error.

[cydrobolt]

will look into expanding the limit, and/or giving an error