Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add authz support for ics20 #3079

Merged
merged 24 commits into from
Feb 2, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
d1b8a69
feat: adding authz support for ICS-20 `MsgTransfer` (#2795)
zmanian Dec 19, 2022
015bce1
chore: moving authz protos to transfer.v1 (#2984)
damiannolan Jan 9, 2023
49f8112
Make isAllowedAddress private (#2985)
chatton Jan 9, 2023
3ffc225
chore: renaming transfer_authorization files (#2990)
chatton Jan 9, 2023
9a2fe70
Change ErrDuplicateEntry to ErrInvalidAuthorization & change error fo…
chatton Jan 9, 2023
2945225
chore: rename transfer authz protobuf types (#2989)
damiannolan Jan 11, 2023
f9be376
chore: refactor NewTransferAuthorization constructor to use varargs (…
damiannolan Jan 11, 2023
aabb07a
e2e: added happy path e2e test for ICS20 authz integration (#2996)
chatton Jan 11, 2023
a1e84c9
chore: implement transfer authz ValidateBasic tests (#2992)
damiannolan Jan 12, 2023
dd1aa5a
chore: refactor transfer authz tests to use table tests (#2998)
damiannolan Jan 16, 2023
05ab435
Add MsgRevoke to ICS20 Authz test (#2997)
chatton Jan 16, 2023
b4bd0a7
chore: TransferAuthorization ValidateBasic now returns an error when …
chatton Jan 16, 2023
ee9db57
E2E Test for spend limit and invalid addresses (#3000)
chatton Jan 18, 2023
048a4b6
Merge branch 'main' into feat/authz-ics20
crodriguezvega Jan 29, 2023
b7e229e
fix v7 import paths
crodriguezvega Jan 29, 2023
d03147e
fix import path
crodriguezvega Jan 29, 2023
7d295c4
replace usage of Bech32Address with FormattedAddress
crodriguezvega Jan 29, 2023
64c0853
Merge branch 'main' into feat/authz-ics20
chatton Jan 30, 2023
49c2a3d
Merge branch 'main' into feat/authz-ics20
chatton Feb 1, 2023
0e9f475
chore: replace use of exp slices package with in line deletion
chatton Feb 1, 2023
3f8040c
Merge branch 'main' into feat/authz-ics20
chatton Feb 1, 2023
024fbb9
Merge branch 'main' into feat/authz-ics20
chatton Feb 1, 2023
c07bf5e
Merge branch 'main' into feat/authz-ics20
chatton Feb 1, 2023
1db27a8
chore: fixing interchaintest import
chatton Feb 1, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/e2e-manual-simd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ on:
- TestInterchainAccountsGroupsTestSuite
- TestInterchainAccountsGovTestSuite
- TestIncentivizedInterchainAccountsTestSuite
- TestAuthzTransferTestSuite
chain-image:
description: 'The image to use for chain A'
required: true
Expand Down
54 changes: 54 additions & 0 deletions docs/ibc/proto-docs.md
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,10 @@

- [Msg](#ibc.applications.transfer.v1.Msg)

- [ibc/applications/transfer/v2/authz.proto](#ibc/applications/transfer/v2/authz.proto)
- [PortChannelAmount](#ibc.applications.transfer.v2.PortChannelAmount)
- [TransferAuthorization](#ibc.applications.transfer.v2.TransferAuthorization)

- [ibc/applications/transfer/v2/packet.proto](#ibc/applications/transfer/v2/packet.proto)
- [FungibleTokenPacketData](#ibc.applications.transfer.v2.FungibleTokenPacketData)

Expand Down Expand Up @@ -2296,6 +2300,56 @@ Msg defines the ibc/transfer Msg service.



<a name="ibc/applications/transfer/v2/authz.proto"></a>
<p align="right"><a href="#top">Top</a></p>

## ibc/applications/transfer/v2/authz.proto



<a name="ibc.applications.transfer.v2.PortChannelAmount"></a>

### PortChannelAmount



| Field | Type | Label | Description |
| ----- | ---- | ----- | ----------- |
| `source_port` | [string](#string) | | the port on which the packet will be sent |
| `source_channel` | [string](#string) | | the channel by which the packet will be sent |
| `spend_limit` | [cosmos.base.v1beta1.Coin](#cosmos.base.v1beta1.Coin) | repeated | spend limitation on the channel |
| `allowed_addresses` | [string](#string) | repeated | |






<a name="ibc.applications.transfer.v2.TransferAuthorization"></a>

### TransferAuthorization
TransferAuthorization allows the grantee to spend up to spend_limit coins from
the granter's account for ibc transfer on a specific channel


| Field | Type | Label | Description |
| ----- | ---- | ----- | ----------- |
| `allocations` | [PortChannelAmount](#ibc.applications.transfer.v2.PortChannelAmount) | repeated | port and channel amounts |





<!-- end messages -->

<!-- end enums -->

<!-- end HasExtensions -->

<!-- end services -->



<a name="ibc/applications/transfer/v2/packet.proto"></a>
<p align="right"><a href="#top">Top</a></p>

Expand Down
8 changes: 8 additions & 0 deletions e2e/testconfig/testconfig.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import (
"encoding/json"
"fmt"
"os"
"strings"

"github.com/cosmos/cosmos-sdk/codec"
sdk "github.com/cosmos/cosmos-sdk/types"
Expand Down Expand Up @@ -132,6 +133,13 @@ func GetChainBTag() string {
return chainBTag
}

// IsCI returns true if the tests are running in CI, false is returned
// if the tests are running locally.
// Note: github actions passes a CI env value of true by default to all runners.
func IsCI() bool {
return strings.ToLower(os.Getenv("CI")) == "true"
}

// ChainOptions stores chain configurations for the chains that will be
// created for the tests. They can be modified by passing ChainOptionConfiguration
// to E2ETestSuite.GetChains.
Expand Down
Loading