Browse Source

admin: require CSRF for S3 tables bucket writes

admin/csrf-s3tables
Chris Lu 11 hours ago
parent
commit
263b7bbf53
  1. 6
      weed/admin/dash/s3tables_management.go

6
weed/admin/dash/s3tables_management.go

@ -579,6 +579,9 @@ func (s *AdminServer) ListS3TablesBucketsAPI(c *gin.Context) {
} }
func (s *AdminServer) CreateS3TablesBucket(c *gin.Context) { func (s *AdminServer) CreateS3TablesBucket(c *gin.Context) {
if !requireSessionCSRFToken(c) {
return
}
var req struct { var req struct {
Name string `json:"name"` Name string `json:"name"`
Tags map[string]string `json:"tags"` Tags map[string]string `json:"tags"`
@ -664,6 +667,9 @@ func (s *AdminServer) SetTableBucketOwner(ctx context.Context, bucketName, owner
} }
func (s *AdminServer) DeleteS3TablesBucket(c *gin.Context) { func (s *AdminServer) DeleteS3TablesBucket(c *gin.Context) {
if !requireSessionCSRFToken(c) {
return
}
bucketArn := c.Query("bucket") bucketArn := c.Query("bucket")
if bucketArn == "" { if bucketArn == "" {
c.JSON(400, gin.H{"error": "Bucket ARN is required"}) c.JSON(400, gin.H{"error": "Bucket ARN is required"})

Loading…
Cancel
Save