chore: added new field account-id with uint64 in AccountAddressByID

CHANGELOG.md

@@ -114,6 +114,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * (x/gov) [#12771](https://github.com/cosmos/cosmos-sdk/pull/12771) Initial deposit requirement for proposals at submission time.
 * (x/staking) [#12967](https://github.com/cosmos/cosmos-sdk/pull/12967) `unbond` now creates only one unbonding delegation entry when multiple unbondings exist at a single height (e.g. through multiple messages in a transaction).
 * (x/auth/vesting) [#13502](https://github.com/cosmos/cosmos-sdk/pull/13502) Add Amino Msg registration for `MsgCreatePeriodicVestingAccount`.
+* (x/auth)[#13780](https://github.com/cosmos/cosmos-sdk/pull/13780) `id` (type of int64) in `AccountAddressByID` grpc query is now deprecated, update to use account-id(type of uint64) to request `AccountAddressByID`.
 * (x/group) [#13876](https://github.com/cosmos/cosmos-sdk/pull/13876) Fix group MinExecutionPeriod that is checked on execution now, instead of voting period end.
 
 ### API Breaking Changes
@@ -171,6 +172,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 extension interfaces. `module.Manager.Modules` is now of type `map[string]interface{}` to support in parallel the new 
 `cosmossdk.io/core/appmodule.AppModule` API.
 * (x/group) [#13876](https://github.com/cosmos/cosmos-sdk/pull/13876) Add `GetMinExecutionPeriod` method on DecisionPolicy interface.
+* (x/auth)[#13780](https://github.com/cosmos/cosmos-sdk/pull/13780) querying with `id` (type of int64) in `AccountAddressByID` grpc query is now throws error, update to use account-id(type of uint64) to request `AccountAddressByID`.
 
 ### CLI Breaking Changes
 

proto/cosmos/auth/v1beta1/query.proto

@@ -197,10 +197,19 @@ message AddressStringToBytesResponse {
 //
 // Since: cosmos-sdk 0.46.2
 message QueryAccountAddressByIDRequest {
+  // Deprecated, use account_id instead
+  // 
   // id is the account number of the address to be queried. This field
   // should have been an uint64 (like all account numbers), and will be
   // updated to uint64 in a future version of the auth query.
-  int64 id = 1;
+  int64 id = 1 [deprecated = true];
+
+  // account_id is the account number of the address to be queried. This field
+  // should have been an uint64 (like all account numbers), and will be
+  // updated to uint64 in a future version of the auth query.
+  // 
+  // Since: cosmos-sdk 0.47
+  uint64 account_id = 2;
 }
 
 // QueryAccountAddressByIDResponse is the response type for AccountAddressByID rpc method

x/auth/client/cli/query.go

@@ -141,13 +141,16 @@ func GetAccountAddressByIDCmd() *cobra.Command {
 				return err
 			}
 
-			accNum, err := strconv.ParseInt(args[0], 10, 64)
+			accNum, err := strconv.ParseUint(args[0], 10, 64)
 			if err != nil {
 				return err
 			}
 
 			queryClient := types.NewQueryClient(clientCtx)
-			res, err := queryClient.AccountAddressByID(cmd.Context(), &types.QueryAccountAddressByIDRequest{Id: accNum})
+			res, err := queryClient.AccountAddressByID(cmd.Context(), &types.QueryAccountAddressByIDRequest{
+				AccountId: accNum,
+			})
+
 			if err != nil {
 				return err
 			}

x/auth/keeper/grpc_query.go

@@ -24,12 +24,16 @@ func (ak AccountKeeper) AccountAddressByID(c context.Context, req *types.QueryAc
 		return nil, status.Errorf(codes.InvalidArgument, "empty request")
 	}
 
-	if req.Id < 0 {
-		return nil, status.Error(codes.InvalidArgument, "invalid account number")
+	var accId uint64
+	switch {
+	case req.Id > 0: //ignoring `0` case since it is default value.
+		return nil, status.Errorf(codes.InvalidArgument, "requesting with id isn't supported, try to request using account-id")
+	case req.AccountId > 0:
+		accId = req.AccountId
 	}
 
 	ctx := sdk.UnwrapSDKContext(c)
-	address := ak.GetAccountAddressByID(ctx, uint64(req.GetId()))
+	address := ak.GetAccountAddressByID(ctx, accId)
 	if len(address) == 0 {
 		return nil, status.Errorf(codes.NotFound, "account address not found with account number %d", req.Id)
 	}

x/auth/keeper/grpc_query_test.go

@@ -174,6 +174,22 @@ func (suite *KeeperTestSuite) TestGRPCQueryAccountAddressByID() {
 			false,
 			func(res *types.QueryAccountAddressByIDResponse) {},
 		},
+		{
+			"invalid: account-id, id are not same",
+			func() {
+				req = &types.QueryAccountAddressByIDRequest{AccountId: 0, Id: 1}
+			},
+			false,
+			func(res *types.QueryAccountAddressByIDResponse) {},
+		},
+		{
+			"invalid: account-id, id are not same",
+			func() {
+				req = &types.QueryAccountAddressByIDRequest{AccountId: 1, Id: -1}
+			},
+			false,
+			func(res *types.QueryAccountAddressByIDResponse) {},
+		},
 		{
 			"account address not found",
 			func() {
@@ -182,6 +198,18 @@ func (suite *KeeperTestSuite) TestGRPCQueryAccountAddressByID() {
 			false,
 			func(res *types.QueryAccountAddressByIDResponse) {},
 		},
+		{
+			"valid account-id",
+			func() {
+				account := suite.accountKeeper.NewAccountWithAddress(suite.ctx, addr)
+				suite.accountKeeper.SetAccount(suite.ctx, account)
+				req = &types.QueryAccountAddressByIDRequest{AccountId: account.GetAccountNumber()}
+			},
+			true,
+			func(res *types.QueryAccountAddressByIDResponse) {
+				suite.Require().NotNil(res.AccountAddress)
+			},
+		},
 		{
 			"valid request",
 			func() {