rainycape / gondola

The web framework for writing faster sites, faster
http://gondolaweb.com
Mozilla Public License 2.0
312 stars 26 forks source link

randomString infinite loop #6

Closed dchest closed 9 years ago

dchest commented 9 years ago

The old version of a function from uniuri in your code (randomString) can lead to infinite loop when number of chars in alphabet is 256, please copy the latest version: https://github.com/dchest/uniuri/blob/master/uniuri.go#L48

(The case isn't triggered by current code due to the fixed alphabet of len < 256, but it's a good thing to fix this anyway.)