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

Jump to bottom

[BUG] ajax.ts is using string trim() which is not supported on IE7/8 #1251

Closed
MSNev opened this issue Apr 15, 2020 · 2 comments · Fixed by #1266
Closed

[BUG] ajax.ts is using string trim() which is not supported on IE7/8 #1251

MSNev opened this issue Apr 15, 2020 · 2 comments · Fixed by #1266
Assignees
@MSNev
Labels
bug es3 ES3 Issues (IE8)
Milestone
2.5.5

Comments

@MSNev
Copy link
Collaborator

MSNev commented Apr 15, 2020

ajax.ts is using string trim() for headers.trim()

This will cause an exception and potential lockup on IE7/8 mode.
@MSNev MSNev self-assigned this Apr 15, 2020
@MSNev MSNev added the bug label Apr 15, 2020
@MSNev MSNev added the es3 ES3 Issues (IE8) label Apr 23, 2020
MSNev pushed a commit that referenced this issue May 13, 2020
[BUG] Can include Correlation Header on IE can fail #1260
74c9988 
[BUG] ajax.ts is using string trim() which is not supported on IE7/8 #1251
@MSNev MSNev mentioned this issue May 13, 2020
Merged
MSNev added a commit that referenced this issue May 20, 2020
@MSNev
[BUG] Can include Correlation Header on IE can fail #1260 (#1266)
7b9f238 
[BUG] ajax.ts is using string trim() which is not supported on IE7/8 #1251
@MSNev MSNev added the fixed - waiting release PR Committed and waiting deployment label May 21, 2020
@MSNev MSNev added this to the 2.5.5 milestone Jun 2, 2020
@MSNev MSNev linked a pull request Jun 2, 2020 that will close this issue
[BUG] Can't include Correlation Header on IE can fail #1260 #1266
Merged
@MSNev MSNev mentioned this issue Jun 2, 2020
Merged
@MSNev MSNev removed the fixed - waiting release PR Committed and waiting deployment label Jun 4, 2020
@MSNev
Copy link
Collaborator Author

MSNev commented Jun 4, 2020

Included in v 2.5.5 which is now fully deployed

@MSNev MSNev closed this as completed Jun 4, 2020
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@MSNev MSNev

Labels
bug es3 ES3 Issues (IE8)
Projects
None yet
Milestone
2.5.5
Development

Successfully merging a pull request may close this issue.

[BUG] Can't include Correlation Header on IE can fail #1260 microsoft/ApplicationInsights-JS
1 participant
@MSNev