internal/pkg: remove typeflag promotion loop
All checks were successful
Test / Create distribution (push) Successful in 47s
Test / Sandbox (push) Successful in 2m49s
Test / ShareFS (push) Successful in 4m54s
Test / Sandbox (race detector) (push) Successful in 5m21s
Test / Hpkg (push) Successful in 5m26s
Test / Hakurei (race detector) (push) Successful in 7m48s
Test / Hakurei (push) Successful in 4m11s
Test / Flake checks (push) Successful in 1m46s

Expanding this enables sharing of code common between types.

Signed-off-by: Ophestra <cat@gensokyo.uk>
This commit is contained in:
2026-01-13 00:23:14 +09:00
parent 5fe166a4a7
commit 156dd767ef

View File

@@ -11,7 +11,6 @@ import (
"io/fs" "io/fs"
"net/http" "net/http"
"os" "os"
"path"
"hakurei.app/container/check" "hakurei.app/container/check"
) )
@@ -142,25 +141,26 @@ func (a *tarArtifact) Cure(t *TContext) (err error) {
r := tar.NewReader(tr) r := tar.NewReader(tr)
for header, err = r.Next(); err == nil; header, err = r.Next() { for header, err = r.Next(); err == nil; header, err = r.Next() {
typeflag := header.Typeflag typeflag := header.Typeflag
for { if typeflag == 0 {
switch typeflag {
case 0:
if len(header.Name) > 0 && header.Name[len(header.Name)-1] == '/' { if len(header.Name) > 0 && header.Name[len(header.Name)-1] == '/' {
typeflag = tar.TypeDir typeflag = tar.TypeDir
} else { } else {
typeflag = tar.TypeReg typeflag = tar.TypeReg
} }
continue
case tar.TypeReg:
p := temp.Append(header.Name).String()
if err = os.MkdirAll(path.Dir(p), 0700); err != nil {
return
} }
pathname := temp.Append(header.Name)
if typeflag >= '0' && typeflag <= '9' && typeflag != tar.TypeDir {
if err = os.MkdirAll(pathname.Dir().String(), 0700); err != nil {
return
}
}
switch typeflag {
case tar.TypeReg:
var f *os.File var f *os.File
if f, err = os.OpenFile( if f, err = os.OpenFile(
p, pathname.String(),
os.O_CREATE|os.O_EXCL|os.O_WRONLY, os.O_CREATE|os.O_EXCL|os.O_WRONLY,
header.FileInfo().Mode()&0500, header.FileInfo().Mode()&0500,
); err != nil { ); err != nil {
@@ -175,49 +175,33 @@ func (a *tarArtifact) Cure(t *TContext) (err error) {
break break
case tar.TypeLink: case tar.TypeLink:
p := temp.Append(header.Name).String() if err = os.Link(header.Linkname, pathname.String()); err != nil {
if err = os.MkdirAll(path.Dir(p), 0700); err != nil {
return
}
if err = os.Link(header.Linkname, p); err != nil {
return return
} }
break break
case tar.TypeSymlink: case tar.TypeSymlink:
p := temp.Append(header.Name).String() if err = os.Symlink(header.Linkname, pathname.String()); err != nil {
if err = os.MkdirAll(path.Dir(p), 0700); err != nil {
return
}
if err = os.Symlink(header.Linkname, p); err != nil {
return return
} }
break break
case tar.TypeDir: case tar.TypeDir:
pathname := temp.Append(header.Name)
madeDirectories = append(madeDirectories, dirTargetPerm{ madeDirectories = append(madeDirectories, dirTargetPerm{
path: pathname, path: pathname,
mode: header.FileInfo().Mode(), mode: header.FileInfo().Mode(),
}) })
if err = os.MkdirAll( if err = os.MkdirAll(pathname.String(), 0700); err != nil {
pathname.String(),
0700,
); err != nil {
return return
} }
break break
case tar.TypeXGlobalHeader: case tar.TypeXGlobalHeader:
// ignore continue // ignore
break
default: default:
return DisallowedTypeflagError(typeflag) return DisallowedTypeflagError(typeflag)
} }
break
}
} }
if errors.Is(err, io.EOF) { if errors.Is(err, io.EOF) {
err = nil err = nil