Skip to content

Fix up of: IA2: Do not treat huge base64 data as NVDA might freeze in Google Chrome (PR #10240)#10282

Merged
michaelDCurran merged 2 commits into
nvaccess:betafrom
accessolutions:pr10240-base64Fixup
Sep 30, 2019
Merged

Fix up of: IA2: Do not treat huge base64 data as NVDA might freeze in Google Chrome (PR #10240)#10282
michaelDCurran merged 2 commits into
nvaccess:betafrom
accessolutions:pr10240-base64Fixup

Conversation

@JulienCochuyt

Copy link
Copy Markdown
Contributor

Link to issue number:

Fix up of PR #10240
Backport of fix up #10281 into beta

Summary of the issue:

As mentioned by @michaelDCurran in #10231 (comment) IAccessibleHandler.splitIA2Attribs might also receive IAccessibleText as argument.

Description of how this pull request fixes the issue:

Alter the regex as to ensure to truncate base64 data only in the src attribute, and not in an eventual
text content (eg. in and inline editor).

Testing performed:

Tested against the test case of #10227.

Known issues with pull request:

Change log entry:

Section: New features, Changes, Bug fixes

… Google Chrome (PR nvaccess#10240)

Ensure to truncate base64 data only in the `src` attribute, and not in an eventual
text content (eg. in and inline editor).

Re: nvaccess#10231 (comment)
@dpy013

dpy013 commented Sep 27, 2019

Copy link
Copy Markdown
Contributor

Link to issue number:

Fix up of PR #10240
Backport of fix up #10281 into beta

Summary of the issue:

As mentioned by @michaelDCurran in #10231 (comment) IAccessibleHandler.splitIA2Attribs might also receive IAccessibleText as argument.

Description of how this pull request fixes the issue:

Alter the regex as to ensure to truncate base64 data only in the src attribute, and not in an eventual
text content (eg. in and inline editor).

Testing performed:

Tested against the test case of #10227.

Known issues with pull request:

Change log entry:

Section: New features, Changes, Bug fixes

The current pr has exceeded the version range of 2019.2.1
thank

@JulienCochuyt

JulienCochuyt commented Sep 27, 2019

Copy link
Copy Markdown
Contributor Author

The current pr has exceeded the version range of 2019.2.1
thank

Indeed, as clarified by @michaelDCurran in 19486b0 this change is worthwhile in master but there is no critical misbehavior in the current implementation, so I guess we won't take the risk of an emergency fix into beta.

Thanks

@michaelDCurran michaelDCurran merged commit 2ed0f47 into nvaccess:beta Sep 30, 2019
@nvaccessAuto nvaccessAuto added this to the 2019.3 milestone Sep 30, 2019
@michaelDCurran michaelDCurran modified the milestones: 2019.3, 2019.2.1 Sep 30, 2019
@JulienCochuyt JulienCochuyt deleted the pr10240-base64Fixup branch September 30, 2019 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants