mirror of
https://github.com/seaweedfs/seaweedfs.git
synced 2026-09-02 06:07:24 +00:00
Chris LuandGitHub
45bf3ad058
* shell: add s3.user.* and s3.policy.attach|detach commands Add focused IAM shell commands following a noun-verb model: - s3.user.create: create user with auto-generated or explicit credentials - s3.user.list: tabular listing with status, policies, key count - s3.user.show: detailed user view (status, source, policies, credentials) - s3.user.delete: delete a user - s3.user.enable: enable a disabled user - s3.user.disable: disable a user (preserves credentials and policies) - s3.policy.attach: attach a named policy to a user - s3.policy.detach: detach a policy from a user These commands are thin wrappers over the existing IAM gRPC service, producing human-readable output instead of raw protobuf text. This is part of a larger effort to replace the monolithic s3.configure command with a composable set of single-purpose commands. * shell: address review feedback for s3.user.* and s3.policy.attach|detach - Return flag parse errors instead of swallowing them (all commands) - Use GetConfiguration instead of N+1 GetUser calls in s3.user.list - Add nil check for resp.Identity in s3.user.show - Fix GetPolicy error masking in s3.policy.attach (wrap original error) - Simplify joinMax using strings.Join * shell: add nil identity guards and wrap gRPC errors - Add nil check for resp.Identity in policy_attach, policy_detach, user_enable, user_disable - Wrap GetUser errors with user context for better diagnostics
120 lines
2.7 KiB
Go
120 lines
2.7 KiB
Go
package shell
|
|
|
|
import (
|
|
"context"
|
|
"flag"
|
|
"fmt"
|
|
"io"
|
|
"strings"
|
|
"time"
|
|
|
|
"github.com/seaweedfs/seaweedfs/weed/pb"
|
|
"github.com/seaweedfs/seaweedfs/weed/pb/iam_pb"
|
|
"google.golang.org/grpc"
|
|
)
|
|
|
|
func init() {
|
|
Commands = append(Commands, &commandS3UserShow{})
|
|
}
|
|
|
|
type commandS3UserShow struct {
|
|
}
|
|
|
|
func (c *commandS3UserShow) Name() string {
|
|
return "s3.user.show"
|
|
}
|
|
|
|
func (c *commandS3UserShow) Help() string {
|
|
return `show details of an S3 IAM user
|
|
|
|
s3.user.show -name <username>
|
|
`
|
|
}
|
|
|
|
func (c *commandS3UserShow) HasTag(CommandTag) bool {
|
|
return false
|
|
}
|
|
|
|
func (c *commandS3UserShow) Do(args []string, commandEnv *CommandEnv, writer io.Writer) error {
|
|
f := flag.NewFlagSet(c.Name(), flag.ContinueOnError)
|
|
name := f.String("name", "", "user name")
|
|
if err := f.Parse(args); err != nil {
|
|
return err
|
|
}
|
|
|
|
if *name == "" {
|
|
return fmt.Errorf("-name is required")
|
|
}
|
|
|
|
return pb.WithGrpcClient(false, 0, func(conn *grpc.ClientConn) error {
|
|
client := iam_pb.NewSeaweedIdentityAccessManagementClient(conn)
|
|
ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
|
|
defer cancel()
|
|
|
|
resp, err := client.GetUser(ctx, &iam_pb.GetUserRequest{Username: *name})
|
|
if err != nil {
|
|
return err
|
|
}
|
|
id := resp.Identity
|
|
if id == nil {
|
|
return fmt.Errorf("user %q returned empty identity", *name)
|
|
}
|
|
|
|
status := "enabled"
|
|
if id.Disabled {
|
|
status = "disabled"
|
|
}
|
|
source := "dynamic"
|
|
if id.IsStatic {
|
|
source = "static"
|
|
}
|
|
|
|
fmt.Fprintf(writer, "Name: %s\n", id.Name)
|
|
fmt.Fprintf(writer, "Status: %s\n", status)
|
|
fmt.Fprintf(writer, "Source: %s\n", source)
|
|
|
|
if id.Account != nil {
|
|
if id.Account.Id != "" {
|
|
fmt.Fprintf(writer, "Account: %s", id.Account.Id)
|
|
if id.Account.DisplayName != "" {
|
|
fmt.Fprintf(writer, " (%s)", id.Account.DisplayName)
|
|
}
|
|
fmt.Fprintln(writer)
|
|
}
|
|
if id.Account.EmailAddress != "" {
|
|
fmt.Fprintf(writer, "Email: %s\n", id.Account.EmailAddress)
|
|
}
|
|
}
|
|
|
|
if len(id.PolicyNames) > 0 {
|
|
fmt.Fprintf(writer, "Policies: %s\n", strings.Join(id.PolicyNames, ", "))
|
|
} else {
|
|
fmt.Fprintln(writer, "Policies: (none)")
|
|
}
|
|
|
|
if len(id.Actions) > 0 {
|
|
fmt.Fprintf(writer, "Actions: %s\n", strings.Join(id.Actions, ", "))
|
|
}
|
|
|
|
fmt.Fprintln(writer)
|
|
if len(id.Credentials) > 0 {
|
|
fmt.Fprintln(writer, "Credentials:")
|
|
for _, cred := range id.Credentials {
|
|
st := cred.Status
|
|
if st == "" {
|
|
st = "Active"
|
|
}
|
|
fmt.Fprintf(writer, " %s %s\n", cred.AccessKey, st)
|
|
}
|
|
} else {
|
|
fmt.Fprintln(writer, "Credentials: (none)")
|
|
}
|
|
|
|
if len(id.ServiceAccountIds) > 0 {
|
|
fmt.Fprintf(writer, "\nService Accounts: %s\n", strings.Join(id.ServiceAccountIds, ", "))
|
|
}
|
|
|
|
return nil
|
|
}, commandEnv.option.FilerAddress.ToGrpcAddress(), false, commandEnv.option.GrpcDialOption)
|
|
}
|