Skip to content

Add first draft of blind auctions EIP#1815

Closed
corollari wants to merge 12 commits intoethereum:masterfrom
corollari:blind-auction2
Closed

Add first draft of blind auctions EIP#1815
corollari wants to merge 12 commits intoethereum:masterfrom
corollari:blind-auction2

Conversation

@corollari
Copy link
Copy Markdown
Contributor

First draft of an EIP that standardises the interface of blind auction contracts.

Discussion: #1745

axic
axic previously approved these changes May 10, 2019
@axic
Copy link
Copy Markdown
Member

axic commented May 19, 2019

@corollari can you please rebase this on master so it can be merged (some new checks were introduced)?

@corollari
Copy link
Copy Markdown
Contributor Author

sure!

Comment thread EIPS/eip-1815.md Outdated

## Motivation
<!--The motivation is critical for EIPs that want to change the Ethereum protocol. It should clearly explain why the existing protocol specification is inadequate to address the problem that the EIP solves. EIP submissions without sufficient motivation may be rejected outright.-->
Many dApps that use blind auctions have appeared in the last years, each one reinventing the wheel and implementing blind auctions in ways that are conceptually similar but with special quirks and different interfaces, thus making it impossible for a system to interact with all of these without accommodating all these differences. This ERC aims to solve that by standarizing an interface for blind auctions contracts, which will enable the creation of systems that can interact with any blind auctions without requiring specialized code.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add some examples for these "many dapps" (including links to the interfaces they use)?

This can be placed in a section called References or perhaps in Rationale explaining some of the decisions based on those.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, in an earlier version I used to have a list of other projects using blind auctions but I ended up deleting it because it felt like just having a list didn't really add much value. I'll just restore it

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It can be a good way to show the user base and that may help in engaging with those projects.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a list of some projects that implemented blind auctions (all of them with slight differences). Not sure how many I should list so just left it at 5.

Comment thread EIPS/eip-1815.md Outdated
Comment thread EIPS/eip-1815.md
interface BlindAuction {
enum Mode { Open, Auction, Owned, Forbidden, Reveal, NotYetAvailable }

event AuctionStarted(bytes32 indexed hash, uint registrationDate);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be a good idea to comment (perhaps using NatSpec) each of these interfaces as done in other ERCs. See http://eips.ethereum.org/EIPS/eip-721 as an example.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will start working on it

Comment thread EIPS/eip-1815.md Outdated
Comment thread EIPS/eip-1815.md

## Rationale
<!--The rationale fleshes out the specification by describing what motivated the design and why particular design decisions were made. It should describe alternate designs that were considered and related work, e.g. how the feature is supported in other languages. The rationale may also provide evidence of consensus within the community, and should discuss important objections or concerns raised during discussion.-->
The interface proposed has been chosen because it is the one used by the Ethereum Name System, which is the most popular project among the ones that use blind auctions, therefore being the one that has the most systems that integrate with it. By picking this interface we aim at minimizing the total amount of work that will be needed to migrate existing systems to the interface proposed, while maximizing the number of systems that will already be compatible with a contract implementing the interface described.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this is an exact copy of what ENS is using, maybe it would be courteous to include the ENS authors in the author field of the ERC?

Copy link
Copy Markdown
Contributor Author

@corollari corollari May 20, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've previously contacted some of them and they didn't seem interested in being authors but I'll try to contact the others.
Edit: If someone else (from the ENS or not) is interested in authoring this EIP along with me please reach out!

@axic axic added the ERC label May 20, 2019
@corollari
Copy link
Copy Markdown
Contributor Author

Thanks for the feedback.

Comment thread EIPS/eip-1815.md Outdated
@corollari
Copy link
Copy Markdown
Contributor Author

Added another implementation by me that is currently WIP.

Comment thread EIPS/eip-1815.md
Comment thread EIPS/eip-1815.md
@axic axic dismissed their stale review May 22, 2019 17:00

Still wip.

@corollari
Copy link
Copy Markdown
Contributor Author

corollari commented May 29, 2019

I won't be able to work on this PR for the upcoming month.

@github-actions
Copy link
Copy Markdown

There has been no activity on this pull request for two months. It will be closed in a week if no further activity occurs. If you would like to move this EIP forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review.

@github-actions github-actions Bot added the stale label Sep 15, 2020
@corollari
Copy link
Copy Markdown
Contributor Author

There's some more work I'd like to do on this EIP but given that I haven't had the time to commit to it, I agree that the best way forward is to close it for now.

@corollari corollari closed this Sep 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants