T3060
Add a new database function to look up registered aliases by address. This will be used to build an API endpoint.
Details
Details
- Reviewers
PiRK - Group Reviewers
Restricted Project - Commits
- rABC621ed0fb0257: [alias-server] Add new database function to lookup registered aliases by address
npm test
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| apps/alias-server/src/db.js | ||
|---|---|---|
| 202 ↗ | (On Diff #41013) | Just double checking - in the event where a dev writes a custom script that sends a registration tx to the payment address for an alias that is already taken, this wouldn't have been registered in the alias-server in the first place right and carries no risk of duplicates flowing through to this function's output? |
Comment Actions
(I'm assuming that the code that writes to the db is in charge of rejecting already registered aliases with different addresses)
| apps/alias-server/src/db.js | ||
|---|---|---|
| 202 ↗ | (On Diff #41013) | Right -- the database collection at config.database.collections.validAliases is initialized with alias as a unique key, i.e. the database cannot accept multiple entries with the same alias The logic for parsing and determining which is "the" valid alias is handled in alias.js For this function, it's safe to assume we can't get back more than one registered alias object with the same alias. |