View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004185 | Taler | wallet (WebExtension) | public | 2016-02-18 17:07 | 2017-06-06 12:18 |
| Reporter | Florian Dold | Assigned To | Florian Dold | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | tweak | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.3 | Fixed in Version | 0.3 | ||
| Summary | 0004185: abstract over RPC in the wallet | ||||
| Description | The WebEx wallet does a lot of RPC (popup and content script calls things from the wallet living in the background page; wallet should call crypto code living in a web worker). In some places this is done rather ad-hoc (construct JSON, use chrome.runtime.sendMessage and chrome.runtime.onMessage on the other end). We should abstract over this a bit, so we don't have to write as much boilerplate code. | ||||
| Tags | No tags attached. | ||||
| In most places we use wrappers now, to translate from type-unsafe chrome callback APIs to our typesafe async/await APIs. Still, we're using strings in many places for RPCs. We can make our RPCs more typesafe with the "keyof T" type, which allows us to check that a string constant is a valid index into a namespace [1]. IMHO we should wait until this feature hits TypeScript stable, which is about by the end of this year. https://github.com/Microsoft/TypeScript/pull/11929 | |
| Fixed in 613a14c1 | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2016-02-18 17:07 | Florian Dold | New Issue | |
| 2016-02-18 17:07 | Florian Dold | Status | new => assigned |
| 2016-02-18 17:07 | Florian Dold | Assigned To | => Florian Dold |
| 2016-02-19 20:41 | Christian Grothoff | Product Version | => 0.0 |
| 2016-02-19 20:41 | Christian Grothoff | Target Version | => 0.1 |
| 2016-02-19 20:41 | Christian Grothoff | Description Updated | |
| 2016-02-20 01:11 | Christian Grothoff | Severity | minor => tweak |
| 2016-03-20 20:53 | Torsten Grothoff | Issue cloned: 0004292 | |
| 2016-09-27 20:26 | Christian Grothoff | Target Version | 0.1 => 0.2 |
| 2016-11-14 03:10 | Florian Dold | Note Added: 0011491 | |
| 2016-11-14 03:10 | Florian Dold | Target Version | 0.2 => 0.3 |
| 2017-04-08 22:28 | Christian Grothoff | Target Version | 0.3 => |
| 2017-05-30 15:02 | Florian Dold | Product Version | 0.0 => 0.3 |
| 2017-05-31 15:08 | Florian Dold | Note Added: 0012198 | |
| 2017-05-31 15:08 | Florian Dold | Status | assigned => informing |
| 2017-05-31 15:08 | Florian Dold | Resolution | open => fixed |
| 2017-06-02 07:35 | Christian Grothoff | Product Version | 0.3 => git (master) |
| 2017-06-02 07:35 | Christian Grothoff | Fixed in Version | => 0.3 |
| 2017-06-02 07:35 | Christian Grothoff | Target Version | => 0.3 |
| 2017-06-06 12:18 | Christian Grothoff | Status | informing => closed |
| 2023-04-13 18:37 | Florian Dold | Category | wallet (WebExtensions) => wallet (WebExtension) |