Skip to content
This repository has been archived by the owner on Nov 5, 2021. It is now read-only.

Gas Abstraction #150

Merged
merged 9 commits into from
Apr 16, 2018
Merged

Gas Abstraction #150

merged 9 commits into from
Apr 16, 2018

Conversation

3esmit
Copy link
Member

@3esmit 3esmit commented Apr 7, 2018

Replaces #73
Requires #145

Gas Abstraction

Preamble

Idea: 150-gas-abstraction
Title: Gas Abstraction
Status: Draft
Created: 2018-02-01
Requires: 145-identity
Replaces: 073-snt-as-gas

Summary

Enable Status users to pay gas fee with any token that is valueable to society.
Users would sign a message request to contracts which will use user's Identity balance of selected token to refund proportional used gas to a incentivezed ETH owner to include that message in a transction.p

Swarm Participants

@oskarth oskarth self-requested a review April 9, 2018 04:50
Title: Gas Abstraction
Status: In Progress
Created: 2018-02-01
Requires: 145-identity
Copy link
Contributor

Choose a reason for hiding this comment

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

Looking at 145-identity it looks like there's a circular dependency which isn't ideal. Can we separate these out somehow?

Copy link
Member Author

Choose a reason for hiding this comment

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

I guess gas-abstraction is dependent of identity, but identity not.
However in a UX aspect, gas-abstraction enhances Identity.

Copy link
Member Author

Choose a reason for hiding this comment

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

gas-abstraction depends on Identity, but it could use other contract as base.
WIth this base the contract can start paying fee for people calling it with owner determined parameters, which are verified by a signed msg.
Could be a MultiSigWallet.


### Product Description

Gas Relay node: Status Destkop extension or an independent node can include messages by making automatic transactions calls to smart contracts to earn tokens being offered as gas price refund.
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't understand these things, can you elaborate or link to elaboration? Gas relay node / adaptors.

Copy link
Member Author

Choose a reason for hiding this comment

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

Gas relay node This are nodes that will be listening to messages in swarm, this messages would give them contract defined "token gas" to who includes it in a transaction.

Adaptor is the function in contract that processes the ethereum signed message, allows the call by gas relay node, and pay the gas relay node the gas in tokens controlled by this contract.


### Requirements & Dependancies

Idea [145-identity](https://github.com/status-im/ideas/pull/145) is important to seamless and safe integration of gas abstraction for any type of call.
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't understand this dependency, could you elaborate?

Copy link
Member Author

Choose a reason for hiding this comment

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

SNTController cannot call another contract in behalf of token holder, because when SNTController calls other contract it becomes msg.sender.
Identity becoming the msg.sender is intended, so Identity can act as an account, and gas relay can use its storage of tokens to pay for gas relayers.

@oskarth
Copy link
Contributor

oskarth commented Apr 12, 2018

@3esmit given conversation in Slack, can we change this to draft, add to README registry and merge it?

@3esmit
Copy link
Member Author

3esmit commented Apr 12, 2018

@oskarth Done! 👍

@oskarth
Copy link
Contributor

oskarth commented Apr 16, 2018

Cool, will add to README in separate commit

@oskarth oskarth merged commit 0d8a151 into master Apr 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants