Make maxUrlLength truncate loop slightly safer/more defensive #918

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Merged

LewisJEllis merged 1 commit into master from url-truncate-fix

Mar 31, 2017

Contributor

LewisJEllis commented Mar 31, 2017

ITT we wish for Array.prototype.forEach.


          Make maxUrlLength truncate loop slightly safer/more defensive

28b66ba

benvinegar approved these changes

View reviewed changes

src/raven.js

                           crumb,
                           data;
-                      for (var i = 0; i < breadcrumbs.values.length; i++) {
+                      for (var i = 0; i < breadcrumbs.values.length; ++i) {

Contributor

benvinegar Mar 31, 2017

ಠ_ಠ

LewisJEllis merged commit 249341b into master

LewisJEllis deleted the url-truncate-fix branch

March 31, 2017 21:59

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet