[token-server] Bump icon upload byte ceiling
ClosedPublic
Actions

Authored by bytesofman on Mon, Dec 2, 17:16.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABCe5732239c6da: [token-server] Bump icon upload byte ceiling

Summary

Does not seem to be happening often, as Cashtab resizes icons to 512x512 before upload, but seems like some users are hitting this.

Bump 1mb size limit to 2mb.

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

token-server-icon-limit-bump

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 31477
Build 62451: Build Diff	token-server-tests
Build 62450: arc lint + arc unit

Event Timeline

bytesofman created this revision.Mon, Dec 2, 17:16

Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Dec 2, 17:16

bytesofman published this revision for review.Mon, Dec 2, 17:16

Harbormaster completed remote builds in B31477: Diff 51335.Mon, Dec 2, 17:18

emack accepted this revision.Tue, Dec 3, 07:30

This revision is now accepted and ready to land.Tue, Dec 3, 07:30

Closed by commit rABCe5732239c6da: [token-server] Bump icon upload byte ceiling (authored by bytesofman). · Explain WhyTue, Dec 3, 14:06

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCe5732239c6da: [token-server] Bump icon upload byte ceiling.

Revision Contents
Changeset List

Path

Size

apps/

token-server/

config.ts

2 lines

Diff 51335

View Options

apps/token-server/config.ts

[token-server] Bump icon upload byte ceilingClosedPublicActions