Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Fixed about pages cannot be synced because data is too big #8023

Closed
luixxiul opened this issue Apr 1, 2017 · 2 comments · Fixed by #8053
Closed

Fixed about pages cannot be synced because data is too big #8023

luixxiul opened this issue Apr 1, 2017 · 2 comments · Fixed by #8053

Comments

@luixxiul
Copy link
Contributor

luixxiul commented Apr 1, 2017

Old title: About pages cannot be synced because data is too big (favicon is encoded in Base64 format) #8023 (comment)

Test plan

#8053 (comment)


Describe the issue you encountered:

(BLESSED_EXTENSION context for cjnmeadmgmiihncdidmfiabhenbggfjm) chrome-extension://cjnmeadmgmiihncdidmfiabhenbggfjm/content/scripts/sync.js:3: Uncaught Error: Data is too big; part limit is 64.{Error: Data is too big; part limit is 64.
    at Object.module.exports.encodeDataToS3KeyArray (chrome-extension://cjnmeadmgmiihncdidmfiabhenbggfjm/content/scripts/sync.js:3:28981)
    at RequestUtil.put (chrome-extension://cjnmeadmgmiihncdidmfiabhenbggfjm/content/scripts/sync.js:1:13433)
    at records.forEach.record (chrome-extension://cjnmeadmgmiihncdidmfiabhenbggfjm/content/scripts/sync.js:2:2197)
    at Array.forEach (native)
    at EventEmitter2.ipc.on (chrome-extension://cjnmeadmgmiihncdidmfiabhenbggfjm/content/scripts/sync.js:2:1873)
    at EventEmitter2.emit (extensions::event_emitter:43:17)
    at EventEmitter2.ipcRenderer.emit (extensions::ipc_utils:31:22)}
  • Platform (Win7, 8, 10? macOS? Linux distro?): Windows 10 32 bit

  • Brave Version (revision SHA): 0.14.1 RC1

  • Steps to reproduce:

    1. Set up sync
    2. Open about:about and bookmark it
    3. Open chrome-extension://cjnmeadmgmiihncdidmfiabhenbggfjm/_generated_background_page.html and wait
  • Actual result: the error above is returned

  • Expected result: either favicon is replaced with a link or data limit is increased to sync the about pages

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    clipboard01

  • Any related issues:

@diracdeltas
Copy link
Member

I think we shouldn't sync data: favicons. Maybe just strip out the favicon for now if it is a data: url.

@diracdeltas diracdeltas added this to the 0.14.2 milestone Apr 3, 2017
@diracdeltas diracdeltas self-assigned this Apr 3, 2017
diracdeltas added a commit that referenced this issue Apr 3, 2017
since they are likely to exceed the AWS upload limit.
fix #8023

Test Plan:
1. unit tests should pass
2. set up sync
3. Open about:about and bookmark it
4. Open chrome-extension://cjnmeadmgmiihncdidmfiabhenbggfjm/_generated_background_page.html and wait. you should not see any errors related to encodeDataToS3KeyArray.
@ayumi
Copy link
Contributor

ayumi commented Apr 3, 2017

the max record size should be ~50 KiB.
however i agree we don't need to sync data URIs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.